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

mill update #14769

Merged
merged 1 commit into from
Dec 18, 2024
Merged

mill update #14769

merged 1 commit into from
Dec 18, 2024

Conversation

patrick-schultz
Copy link
Collaborator

@patrick-schultz patrick-schultz commented Dec 16, 2024

Change Description

Update mill to 0.12.4. Modify the build script to use the new build.mill style, and to move top level tasks into the root module.

Also needed to update scala from 2.12.15 to 2.12.20, for compatibility with the semanticdb compiler plugin used by scalafix.

Security Assessment

  • This change has no security impact

Impact Description

Small refactoring of the build system.

Copy link
Collaborator Author

patrick-schultz commented Dec 16, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

hail/build.mill Show resolved Hide resolved
hail/mill Show resolved Hide resolved
@patrick-schultz patrick-schultz force-pushed the ps-12-16-mill_update branch 2 times, most recently from dcc28c1 to 728ff17 Compare December 17, 2024 18:24
@patrick-schultz patrick-schultz marked this pull request as ready for review December 17, 2024 18:27
Copy link
Collaborator

@chrisvittal chrisvittal left a comment

Choose a reason for hiding this comment

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

I assume that the scala changes are due to the upgrade to scalafix?

Thanks for taking care of this.

Copy link
Collaborator Author

It seemed to be the scala upgrade adding new warnings, particularly for semantic disagreements between 2.x and 3.x, and we have fatal warnings on.

@hail-ci-robot hail-ci-robot merged commit a535fcb into main Dec 18, 2024
3 checks passed
@hail-ci-robot hail-ci-robot deleted the ps-12-16-mill_update branch December 18, 2024 18:28
grohli pushed a commit to grohli/hail that referenced this pull request Jan 27, 2025
## Change Description

Update mill to 0.12.4. Modify the build script to use the new
`build.mill` style, and to move top level tasks into the root module.

Also needed to update scala from 2.12.15 to 2.12.20, for compatibility
with the semanticdb compiler plugin used by scalafix.

## Security Assessment

- This change has no security impact

### Impact Description

Small refactoring of the build system.
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.

4 participants