Skip to content
This repository was archived by the owner on Jan 14, 2026. It is now read-only.

Conversation

@alexeagle
Copy link
Contributor

@alexeagle alexeagle commented Oct 31, 2023

This also fixes our current dependency on GitHub archives which are not guaranteed to be stable, https://github.com/bazelbuild/bazel-central-registry/pull/332/files#diff-7288b02983f19cbe12177d5f193a6bfc533e352fc44fbe12cdb93e1c8c28dcd1R4 which requires a special 'skip-url-stability-check' label on the BCR PR

Instead, we run 'git archive' to create the release artifact. We can now prune the tests/ folder from that artifact, which lets us remove an exclusion from the presubmit as well.

Fixes #188

@alexeagle alexeagle requested review from a team and comius as code owners October 31, 2023 14:57
Copy link
Collaborator

@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

This also fixes our current dependency on GitHub archives which are not guaranteed to be stable,
https://github.com/bazelbuild/bazel-central-registry/pull/332/files#diff-7288b02983f19cbe12177d5f193a6bfc533e352fc44fbe12cdb93e1c8c28dcd1R4
which requires a special 'skip-url-stability-check' label on the BCR PR

Instead, we run 'git archive' to create the release artifact. We can now prune the tests/ folder from that artifact, which lets us remove an exclusion from the presubmit as well.
alexeagle added a commit to aspect-build/rules_esbuild that referenced this pull request Nov 1, 2023
alexeagle added a commit to aspect-build/rules_py that referenced this pull request Nov 1, 2023
alexeagle added a commit to aspect-build/rules_ts that referenced this pull request Nov 1, 2023
alexeagle added a commit to aspect-build/rules_ts that referenced this pull request Nov 1, 2023
alexeagle added a commit to bazel-contrib/rules-template that referenced this pull request Nov 1, 2023
Illustrate how to stamp the version into the release artifact as well, which we've needed in only a couple of rulesets.
Makes it match bazelbuild/rules_proto#189
alexeagle added a commit to bazel-contrib/rules-template that referenced this pull request Nov 1, 2023
Illustrate how to stamp the version into the release artifact as well, which we've needed in only a couple of rulesets.
Makes it match bazelbuild/rules_proto#189
alexeagle added a commit to bazel-contrib/bazel-lib that referenced this pull request Nov 1, 2023
alexeagle added a commit to aspect-build/rules_py that referenced this pull request Nov 1, 2023
Makes it match bazelbuild/rules_proto#189

---

### Type of change

- Refactor (a code change that neither fixes a bug or adds a new
feature)

### Test plan

- Manual testing; please provide instructions so we can reproduce:
I made a clean git repo and verified the export-* settings work in
.gitattributes
alexeagle added a commit to bazel-contrib/bazel-lib that referenced this pull request Nov 1, 2023
alexeagle added a commit to aspect-build/rules_esbuild that referenced this pull request Nov 1, 2023
alexeagle added a commit to aspect-build/rules_ts that referenced this pull request Nov 1, 2023
comius
comius previously approved these changes Nov 2, 2023
@comius comius added the ready to pull Required label for automatic import to Google label Nov 2, 2023
@comius comius merged commit aaa54ca into bazelbuild:main Nov 2, 2023
alexeagle added a commit to bazel-contrib/bazel-lib that referenced this pull request Dec 23, 2023
comius added a commit that referenced this pull request Feb 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ready to pull Required label for automatic import to Google

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automate releases so 'push a tag' is the entire instructions

5 participants