Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Unicode 15.1 version numbers #340

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 5, 2022
Merged

Conversation

markusicu
Copy link
Member

@markusicu markusicu commented Sep 30, 2022

Updating version numbers, with latestVersion=15.1.0.

I also modified GenerateEnums.java to write @Override annotations that Eclipse adds automatically, so that when one touches UcdPropertyValues.java in Eclipse, the CI check does not complain about unexpected diffs.

The second commit makes minor version number changes in generated segmentation .html files.

@markusicu markusicu force-pushed the uni15.1version branch 2 times, most recently from 050c108 to 6ccba6c Compare September 30, 2022 22:30
@markusicu markusicu marked this pull request as ready for review October 4, 2022 21:26
@@ -43,16 +43,16 @@ jobs:
run: >
mkdir -pv $(pwd)/output/Generated/ &&
mvn -s .github/workflows/mvn-settings.xml -B exec:java
-Dexec.mainClass="org.unicode.text.UCD.Main" -Dexec.args="version 15.0.0 build MakeUnicodeFiles"
-Dexec.mainClass="org.unicode.text.UCD.Main" -Dexec.args="version 15.1.0 build MakeUnicodeFiles"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there any way to expand this from the CURRENT_UVERSION token in cli-build-instructions.yml?

Copy link
Member Author

Choose a reason for hiding this comment

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

I suspect not, since it's in a different workflow file. But I am no expert in GitHub Actions.

Copy link
Contributor

Choose a reason for hiding this comment

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

I am entirely unsure myself, but even if we needed to create a new file containing just the constant such that it could be used by the others I think it would be worth doing.

Copy link
Member Author

@markusicu markusicu Oct 5, 2022

Choose a reason for hiding this comment

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

--> issue #344

@markusicu
Copy link
Member Author

I ran the UCD file generator and added a second commit with minor version number changes in generated segmentation .html files.

@markusicu markusicu requested a review from nedley October 5, 2022 20:15
@markusicu markusicu merged commit eb324a3 into unicode-org:main Oct 5, 2022
@markusicu markusicu deleted the uni15.1version branch October 5, 2022 21:27
@markusicu markusicu mentioned this pull request Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants