Skip to content

Conversation

@robertbastian
Copy link
Member

@robertbastian robertbastian commented Dec 1, 2025

@robertbastian robertbastian requested a review from srl295 December 1, 2025 13:04
Copy link
Member

@srl295 srl295 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 with questions

working-directory: cldr-json
run: |
cd cldr-json
echo "::group::Cleanup"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you drop these? they categorize the log lines in the output panel

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because I've made them separate steps, which are a better categorization

exit 1
fi

export MAVEN_OPTS="-Xmx16384m -Dexec.cleanupDaemonThreads=false"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would put this on cldr-config.sh so that it can be overridden but this is fine for now

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it was here before, just further down

@robertbastian robertbastian merged commit b08a82b into main Dec 1, 2025
2 checks passed
@robertbastian robertbastian deleted the pretty-grouse branch December 1, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants