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

Add rpmpack to the BCR repository. #1484

Closed
wants to merge 1 commit into from
Closed

Conversation

jarondl
Copy link

@jarondl jarondl commented Feb 15, 2024

No description provided.

@bazel-io
Copy link
Member

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (rpmpack) have been updated in this PR. Please review the changes.

@meteorcloudy meteorcloudy added the presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval label Feb 15, 2024
@meteorcloudy
Copy link
Member

@jarondl Can you please address the presubmit failures?

@meteorcloudy
Copy link
Member

/cc @alexeagle @aiuto

@meteorcloudy meteorcloudy self-requested a review February 15, 2024 13:46
@keith keith mentioned this pull request Mar 21, 2024
@keith
Copy link
Member

keith commented Mar 21, 2024

I took this PR and submitted a new version with minor fixes to fix CI, please check it out! #1658

keith added a commit that referenced this pull request Mar 21, 2024
Forked from
#1484

Very minor changes compared to the original PR, I fixed the targets in the presubmit.yml, added the missing strip_prefix in the source.json, and put the correct MODULE.bazel contents here since they are already setup correctly upstream
@keith
Copy link
Member

keith commented Mar 21, 2024

mine merged so closing this one. @jarondl if you're interested the biggest change I made was reducing the number of targets being tested in the presubmit since macOS and windows don't have docker to run the docker_diff targets

@keith keith closed this Mar 21, 2024
@alexeagle
Copy link
Contributor

Random drive-by comment - on rules_oci we HAVE to test docker on Macos so we have a Testcontainers API key to get a remote docker daemon

@jarondl
Copy link
Author

jarondl commented Mar 28, 2024

Thank you @keith ! that's super awesome.

aiuto pushed a commit to aiuto/bazel-central-registry that referenced this pull request Jun 3, 2024
Forked from
bazelbuild#1484

Very minor changes compared to the original PR, I fixed the targets in the presubmit.yml, added the missing strip_prefix in the source.json, and put the correct MODULE.bazel contents here since they are already setup correctly upstream
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants