Skip to content
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

Use only one variable for mill-version files #3536

Merged
merged 1 commit into from
Jan 12, 2025

Conversation

fthomas
Copy link
Member

@fthomas fthomas commented Jan 11, 2025

Using one variable is enough since millVersionNameInConfig was a suffix of millVersionName. If Scala Steward is allowed to edit files in ending with mill-version it is also allowed to edit .mill-version files.

Using one variable is enough since `millVersionNameInConfig` was a
suffix of `millVersionName`.
@fthomas fthomas added this to the 0.32.2 milestone Jan 11, 2025
Copy link

codecov bot commented Jan 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.77%. Comparing base (eae30d0) to head (f297f74).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3536      +/-   ##
==========================================
- Coverage   89.78%   89.77%   -0.02%     
==========================================
  Files         171      171              
  Lines        4984     4979       -5     
  Branches      484      496      +12     
==========================================
- Hits         4475     4470       -5     
  Misses        509      509              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fthomas fthomas merged commit 26f9dd0 into main Jan 12, 2025
18 checks passed
@fthomas fthomas deleted the topic/use-one-mill-version-var branch January 12, 2025 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants