Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Manage TestModule dependencies via build.sc #3320

Merged
merged 5 commits into from
Aug 6, 2024
Merged

Conversation

lefou
Copy link
Member

@lefou lefou commented Jul 29, 2024

This allows us to detect possible updates easier.

@lefou lefou changed the title Mange TestModule dependencies via build.sc Bump and manage TestModule dependencies via build.sc Aug 1, 2024
Comment on lines +20 to +28
// plugins and dependencies
import $file.ci.shared
import $file.ci.upload
import $ivy.`org.scalaj::scalaj-http:2.4.2`
import $ivy.`de.tototec::de.tobiasroeser.mill.vcs.version::0.4.0`
import $ivy.`com.github.lolgab::mill-mima::0.1.1`
import $ivy.`net.sourceforge.htmlcleaner:htmlcleaner:2.29`
import $ivy.`com.lihaoyi::mill-contrib-buildinfo:`
import $ivy.`com.goyeau::mill-scalafix::0.4.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.

I moved these after the ordinary imports to work around IntelliJ removing them when "organizing" them. Since we parse them ahead of compilation, it makes no semantic difference.

@lefou lefou marked this pull request as ready for review August 6, 2024 08:34
@lefou lefou changed the title Bump and manage TestModule dependencies via build.sc Manage TestModule dependencies via build.sc Aug 6, 2024
@lefou lefou requested a review from lihaoyi August 6, 2024 13:28
@lefou lefou merged commit f908fca into main Aug 6, 2024
30 checks passed
@lefou lefou deleted the manage-test-dependencies branch August 6, 2024 19:01
@lefou lefou added this to the after 0.11.11 milestone Aug 6, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants