Skip to content

Rename java/modules to java/common#174

Closed
fmeum wants to merge 1 commit into
bazelbuild:masterfrom
fmeum:master
Closed

Rename java/modules to java/common#174
fmeum wants to merge 1 commit into
bazelbuild:masterfrom
fmeum:master

Conversation

@fmeum

@fmeum fmeum commented Mar 8, 2024

Copy link
Copy Markdown
Contributor

The term "modules" is easily confused with "Bazel modules" from Bzlmod.

@fmeum fmeum requested review from a team and comius as code owners March 8, 2024 17:06
@fmeum

fmeum commented Mar 8, 2024

Copy link
Copy Markdown
Contributor Author

CC @hvadehra @comius

@fmeum

fmeum commented Mar 8, 2024

Copy link
Copy Markdown
Contributor Author

We discussed this on the ruleset structure design doc, where common was the preferred alternative.

The term "modules" is easily confused with "Bazel modules" from Bzlmod.
comius pushed a commit to comius/rules_java that referenced this pull request Sep 2, 2024
Copybara Import from bazelbuild#174

BEGIN_PUBLIC

Rename `java/modules` to `java/common` (bazelbuild#174)

The term "modules" is easily confused with "Bazel modules" from Bzlmod.

Closes bazelbuild#174

END_PUBLIC

COPYBARA_INTEGRATE_REVIEW=bazelbuild#174 from fmeum:master 5a9babe
PiperOrigin-RevId: 615070661
Change-Id: I0731edb4f8ce3e6a19a742fe7fd51dbde05bd53b
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.

2 participants