Draft: Add bzlmod minimal support

bazelbuild / rules_scala

Scala rules for Bazel

Apache License 2.0

359 stars 266 forks source link

Draft: Add bzlmod minimal support #1541

Open mateuszkuta256 opened 6 months ago

mateuszkuta256 commented 6 months ago

Description

This is the first step of migration to Bzlmod, see https://bazel.build/external/migration

I'm gonna create a few further pull requests to bring all the changes described here: https://github.com/bazelbuild/rules_scala/issues/1482#issuecomment-1907672400 The migration process will finish once WORKSPACE.bzlmod is empty

Motivation

Resolve https://github.com/bazelbuild/rules_scala/issues/1482

mateuszkuta256 commented 5 months ago

I turn into a draft because will surely cause conflicts with these changes:

SCALA_VERSION dependent code has to be refactored to read this information from the toolchain

simuons commented 5 months ago

Hi, @mateuszkuta256, thanks for taking this. So as I understand this is somewhat "blocked" by required changes to toolchains and SCALA_VERSION. I'll reply in multi scala issue shortly. This change and multi scala are the big ones. So maybe lets first solve multi scala and toolchains and then bzlmod (i have a hunch that proper toolchains would make bzlmod migration easier).