Skip to content

[query] fully define mill compile dependencies#15208

Draft
ehigham wants to merge 1 commit intomainfrom
ehigham/full-defined-builds
Draft

[query] fully define mill compile dependencies#15208
ehigham wants to merge 1 commit intomainfrom
ehigham/full-defined-builds

Conversation

@ehigham
Copy link
Member

@ehigham ehigham commented Dec 9, 2025

Change Description

Fixes #<issue_number> (delete if N/A)

Brief description and justification of what this PR is doing.

Security Assessment

Delete all except the correct answer:

  • This change potentially impacts the Hail Batch instance as deployed by Broad Institute in GCP
    • The Impact Rating, Impact Description, and Appsec Review sections are required
  • This change cannot impact the Hail Batch instance as deployed by Broad Institute in GCP
    • The Impact Rating, Impact Description, and Appsec Review sections can be deleted

Impact Rating

Delete all except the correct answer:

  • This change has a high security impact
  • This change has a medium security impact
  • This change has a low security impact
  • This change has no security impact

Impact Description

Replace this content with a description of the impact of the change:

  • For none/low impact: a quick one/two sentence justification of the rating.
    • Example: "Docs only", "Low-level refactoring of non-security code", etc.
  • For medium/high impact: provide a description of the impact and the mitigations in place.
    • Example: "New UI text field added in analogy to existing elements, with input strings escaped and validated against code injection"

Appsec Review

  • Required: The impact has been assessed and approved by appsec

Copy link
Member Author

ehigham commented Dec 9, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@ehigham ehigham force-pushed the ehigham/full-defined-builds branch from d9d9dc1 to d7bd0b5 Compare December 9, 2025 18:59
@ehigham ehigham force-pushed the ehigham/mill-multi-file-builds branch 2 times, most recently from 12b5c36 to 815138a Compare December 9, 2025 19:01
@ehigham ehigham force-pushed the ehigham/full-defined-builds branch 2 times, most recently from 0e931d5 to 7528b7b Compare December 9, 2025 19:04
@ehigham ehigham force-pushed the ehigham/mill-multi-file-builds branch from 815138a to b294921 Compare December 9, 2025 19:06
@ehigham ehigham force-pushed the ehigham/full-defined-builds branch 2 times, most recently from 2326dab to 7014fe4 Compare December 15, 2025 17:06
@ehigham ehigham force-pushed the ehigham/mill-multi-file-builds branch from b294921 to 25732a0 Compare December 15, 2025 17:06
@ehigham ehigham force-pushed the ehigham/full-defined-builds branch from 7014fe4 to 9a828d3 Compare December 15, 2025 17:44
@ehigham ehigham force-pushed the ehigham/mill-multi-file-builds branch from 25732a0 to bae5d57 Compare December 15, 2025 17:44
Base automatically changed from ehigham/mill-multi-file-builds to main December 15, 2025 19:30
@ehigham ehigham force-pushed the ehigham/full-defined-builds branch 3 times, most recently from 42877f7 to ad27f1d Compare December 16, 2025 17:42
@ehigham ehigham force-pushed the ehigham/full-defined-builds branch 5 times, most recently from 8eebfa3 to 80eba71 Compare January 7, 2026 16:43
@ehigham ehigham force-pushed the ehigham/full-defined-builds branch from 80eba71 to ea325d6 Compare January 8, 2026 15:30
Comment on lines +208 to +213
`scala-collection-compat` :: "2.7.0",
`scala-compiler` :: "2.12.18",
`scala-library` :: "2.12.18",
`scala-parser-combinators` :: "2.3.0",
`scala-reflect` :: "2.12.18",
`scala-xml` :: "2.1.0",
Copy link
Member Author

Choose a reason for hiding this comment

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

These depend on scala version surely?

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.

1 participant