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

Bazel 6 Migration #34

Merged
merged 103 commits into from
Jul 12, 2024
Merged

Bazel 6 Migration #34

merged 103 commits into from
Jul 12, 2024

Conversation

adierkens
Copy link
Member

The updates ruleset to leverage bazel@6, rules_js, bazel modules, vitest and tsup

@adierkens adierkens added major Increment the major version when merged internal Changes only affect the internal API labels Nov 28, 2023
@adierkens adierkens mentioned this pull request Nov 28, 2023
6 tasks
Comment on lines +4 to +29
build --bes_results_url=https://app.buildbuddy.io/invocation/
build --bes_backend=grpcs://remote.buildbuddy.io
build --remote_cache=grpcs://remote.buildbuddy.io
build --experimental_remote_cache_compression
build --experimental_remote_cache_async
build --remote_download_toplevel
build --remote_timeout=3600
build --build_metadata=REPO_URL=https://github.com/player-ui/rules_player.git

test --bes_results_url=https://app.buildbuddy.io/invocation/
test --bes_backend=grpcs://remote.buildbuddy.io
test --remote_cache=grpcs://remote.buildbuddy.io
test --experimental_remote_cache_compression
test --experimental_remote_cache_async
test --remote_download_toplevel
test --remote_timeout=3600
test --build_metadata=REPO_URL=https://github.com/player-ui/rules_player.git

coverage --bes_results_url=https://app.buildbuddy.io/invocation/
coverage --bes_backend=grpcs://remote.buildbuddy.io
coverage --remote_cache=grpcs://remote.buildbuddy.io
coverage --experimental_remote_cache_compression
coverage --experimental_remote_cache_async
coverage --remote_download_toplevel
coverage --remote_timeout=3600
coverage --build_metadata=REPO_URL=https://github.com/player-ui/rules_player.git
Copy link
Member

Choose a reason for hiding this comment

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

Can we collapse these into common:

Suggested change
build --bes_results_url=https://app.buildbuddy.io/invocation/
build --bes_backend=grpcs://remote.buildbuddy.io
build --remote_cache=grpcs://remote.buildbuddy.io
build --experimental_remote_cache_compression
build --experimental_remote_cache_async
build --remote_download_toplevel
build --remote_timeout=3600
build --build_metadata=REPO_URL=https://github.com/player-ui/rules_player.git
test --bes_results_url=https://app.buildbuddy.io/invocation/
test --bes_backend=grpcs://remote.buildbuddy.io
test --remote_cache=grpcs://remote.buildbuddy.io
test --experimental_remote_cache_compression
test --experimental_remote_cache_async
test --remote_download_toplevel
test --remote_timeout=3600
test --build_metadata=REPO_URL=https://github.com/player-ui/rules_player.git
coverage --bes_results_url=https://app.buildbuddy.io/invocation/
coverage --bes_backend=grpcs://remote.buildbuddy.io
coverage --remote_cache=grpcs://remote.buildbuddy.io
coverage --experimental_remote_cache_compression
coverage --experimental_remote_cache_async
coverage --remote_download_toplevel
coverage --remote_timeout=3600
coverage --build_metadata=REPO_URL=https://github.com/player-ui/rules_player.git
common --bes_results_url=https://app.buildbuddy.io/invocation/
common --bes_backend=grpcs://remote.buildbuddy.io
common --remote_cache=grpcs://remote.buildbuddy.io
common --experimental_remote_cache_compression
common --experimental_remote_cache_async
common --remote_download_toplevel
common --remote_timeout=3600
common --build_metadata=REPO_URL=https://github.com/player-ui/rules_player.git

@KetanReddy KetanReddy marked this pull request as ready for review July 11, 2024 23:38
@KetanReddy KetanReddy changed the title [WIP] Bazel 6 Migration Bazel 6 Migration Jul 11, 2024
@KetanReddy KetanReddy merged commit 3c2967e into main Jul 12, 2024
2 checks passed
@KetanReddy KetanReddy deleted the bazel-6 branch July 12, 2024 04:25
@intuit-svc
Copy link

🚀 PR was released in v1.0.0 🚀

@intuit-svc intuit-svc added the released This issue/pull request has been released. label Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Changes only affect the internal API major Increment the major version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants