Skip to content
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

fix(ci): fix bzlmod issues and enable on ci #658

Merged
merged 1 commit into from Nov 15, 2023

Conversation

kormide
Copy link
Member

@kormide kormide commented Nov 15, 2023

--enable_bzlmod was not being set on CI runs. It was attempted for windows, but was passed in the startup flags area. This exposed a bunch of bzlmod issues throughout the repo.

The docs changes are just caused by a newer stardoc version.

This will need to be backported to 1.X.

Type of change

  • Bug fix (change which fixes an issue)

Test plan

  • Covered by existing test cases

@kormide kormide force-pushed the fix-bzlmod-ci branch 4 times, most recently from 11c8c2d to 70d01b7 Compare November 15, 2023 21:34
@kormide kormide force-pushed the fix-bzlmod-ci branch 5 times, most recently from 123d631 to 64ef432 Compare November 15, 2023 22:49
@kormide kormide changed the title fix(ci): enable bzlmod on ci runs fix(ci): fix bzlmod issues and enable on ci Nov 15, 2023
@kormide kormide force-pushed the fix-bzlmod-ci branch 2 times, most recently from 361f73b to 3730129 Compare November 15, 2023 23:00
@kormide kormide marked this pull request as ready for review November 15, 2023 23:00
Copy link
Member

@thesayyn thesayyn left a comment

Choose a reason for hiding this comment

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

LGTM. nice job!

lib/tests/paths_test.bzl Show resolved Hide resolved
lib/tests/tar/BUILD.bazel Show resolved Hide resolved
@kormide kormide merged commit 5bd6e5f into aspect-build:main Nov 15, 2023
24 checks passed
@kormide kormide deleted the fix-bzlmod-ci branch November 15, 2023 23:07
kormide added a commit to kormide/bazel-lib that referenced this pull request Nov 15, 2023
kormide added a commit to kormide/bazel-lib that referenced this pull request Nov 15, 2023
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.

None yet

3 participants