Skip to content

Update URI gem to 1.0.3 in JRuby 10#8751

Merged
headius merged 3 commits intojruby:masterfrom
andsel:fix/update_uri_version_1.0.3
Apr 8, 2025
Merged

Update URI gem to 1.0.3 in JRuby 10#8751
headius merged 3 commits intojruby:masterfrom
andsel:fix/update_uri_version_1.0.3

Conversation

@andsel
Copy link
Contributor

@andsel andsel commented Apr 7, 2025

No description provided.

@andsel andsel changed the title Update URI ge mto 1.0.3 Update URI gem to 1.0.3 Apr 7, 2025
<include>gems/tsort-0.2.0*/**/*</include>
<include>gems/un-0.3.0*/**/*</include>
<include>gems/uri-1.0.2*/**/*</include>
<include>gems/uri-1.0.3*/**/*</include>

Choose a reason for hiding this comment

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

The header of pom.xml says "DO NOT MODIFY - GENERATED CODE". I think this file is not supposed to update.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @kaisecheng , fixed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated executing a build, run ./mvnw

@headius
Copy link
Member

headius commented Apr 7, 2025

Actually we do commit changes to that file but it should not be edited by hand. It gets regenerated by the build, so you make a change to the pom.rb, run one build, and commit both files.

Thanks for the PR! A few things for you to consider:

  • JRuby master is JRuby 10. If this change is only intended for JRuby 10, that's fine... otherwise it should be retargeted against the jruby-9.4 branch.
  • Master should be green now, modulo some intermittent failures we are still catching in CI. If you rebase, it should provide a better view of pass/fail.
  • You can go ahead and restore the changes to the pom.xml.

@andsel andsel changed the base branch from master to jruby-9.4 April 8, 2025 13:23
@andsel andsel changed the base branch from jruby-9.4 to master April 8, 2025 13:25
@andsel
Copy link
Contributor Author

andsel commented Apr 8, 2025

Thanks @headius for checking. This PR is targeted for JRuby 10, while the one for 9.4 branch is #8755

@andsel andsel changed the title Update URI gem to 1.0.3 Update URI gem to 1.0.3 on JRuby 10 Apr 8, 2025
@andsel andsel changed the title Update URI gem to 1.0.3 on JRuby 10 Update URI gem to 1.0.3 in JRuby 10 Apr 8, 2025
@headius headius added this to the JRuby 10.0.0.0 milestone Apr 8, 2025
@headius headius merged commit 34f5906 into jruby:master Apr 8, 2025
72 checks passed
@headius
Copy link
Member

headius commented Apr 8, 2025

Thank you!

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