Skip to content

RATIS-2439. Remove URL from ratis-version.properties#1380

Merged
OneSizeFitsQuorum merged 1 commit intoapache:masterfrom
adoroszlai:RATIS-2439
Mar 18, 2026
Merged

RATIS-2439. Remove URL from ratis-version.properties#1380
OneSizeFitsQuorum merged 1 commit intoapache:masterfrom
adoroszlai:RATIS-2439

Conversation

@adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Version info URL depends on the origin repo which we clone from. Including it in build artifacts may make the build non-reproducible (see discussion).

https://issues.apache.org/jira/browse/RATIS-2439

How was this patch tested?

https://github.com/adoroszlai/ratis/actions/runs/23214264399

@adoroszlai adoroszlai self-assigned this Mar 17, 2026
@adoroszlai adoroszlai requested a review from szetszwo March 17, 2026 21:28
Copy link
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

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

+1 the change looks good.

Copy link
Contributor

@OneSizeFitsQuorum OneSizeFitsQuorum left a comment

Choose a reason for hiding this comment

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

LGTM

@OneSizeFitsQuorum OneSizeFitsQuorum merged commit 3767cf9 into apache:master Mar 18, 2026
16 checks passed
@adoroszlai adoroszlai deleted the RATIS-2439 branch March 18, 2026 06:42
@adoroszlai
Copy link
Contributor Author

Thanks @OneSizeFitsQuorum, @szetszwo for the review.

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