-
Notifications
You must be signed in to change notification settings - Fork 9
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
Migrate to Bzlmod #47
Conversation
MODULE.bazel
Outdated
bazel_dep(name = "rules_jvm_external", version = "6.4") | ||
bazel_dep(name = "rules_scala_annex") | ||
|
||
rules_scala_annex_version = "bzlmod-migration" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Wait until we've merged Migrate to Bzlmod rules_scala#74 and update this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly LGTM. Left a few comments.
MODULE.bazel
Outdated
module(name = "rules_twirl") | ||
|
||
bazel_dep(name = "bazel_skylib", version = "1.7.1") | ||
bazel_dep(name = "buildifier_prebuilt", version = "7.3.1") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can mark this as dev_dependency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comment here:
lucidsoftware/rules_scala#74 (comment)
MODULE.bazel
Outdated
bazel_dep(name = "bazel_skylib", version = "1.7.1") | ||
bazel_dep(name = "buildifier_prebuilt", version = "7.3.1") | ||
bazel_dep(name = "rules_java", version = "7.11.1") | ||
bazel_dep(name = "rules_pkg", version = "1.0.1") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can mark this as dev_dependency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comment here:
lucidsoftware/rules_scala#74 (comment)
MODULE.bazel
Outdated
urls = ["https://github.com/lucidsoftware/rules_scala/archive/refs/heads/{}.zip".format(rules_scala_annex_version)], | ||
) | ||
|
||
bazel_dep(name = "stardoc", version = "0.7.1") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be moved up with the other bazel_deps
?
Also I think we can mark this as dev_dependency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comment here:
lucidsoftware/rules_play_routes#57 (comment)
MODULE.bazel
Outdated
|
||
specs2_version = "4.20.8" | ||
|
||
zinc_version = "1.10.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Scope creep, but can we please update this to 1.10.4? Annex is already on that version, so it should be as simple as changing the version and updating the lockfile.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure thing.
d1f4321
to
20610a7
Compare
Squashed. |
No description provided.