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 verify_archive rule to do e2e tests on built archives. #669

Merged
merged 1 commit into from
Feb 15, 2023

Conversation

aiuto
Copy link
Collaborator

@aiuto aiuto commented Feb 10, 2023

We could do more, but this should be good enough to start.

Let's review this as is for the match, and then I will add zip support in a followup.

Fixes #644

@aiuto aiuto requested a review from nacl as a code owner February 10, 2023 22:49
@aiuto aiuto requested a review from sdtwigg February 10, 2023 22:51
We could do more, but this should be good enough to start.

Fixes bazelbuild#644
Copy link

@sdtwigg sdtwigg left a comment

Choose a reason for hiding this comment

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

docstrings look fine and sufficiently self-explanatory

@aiuto
Copy link
Collaborator Author

aiuto commented Feb 15, 2023

Thanks.

@aiuto
Copy link
Collaborator Author

aiuto commented Feb 15, 2023

Actually, I need an approve review.

@aiuto
Copy link
Collaborator Author

aiuto commented Feb 15, 2023

Oh.. weird, why didn't the approve from you work? I'll go hunt. Stay tuned.

@aiuto aiuto merged commit 3b8e3f1 into bazelbuild:main Feb 15, 2023
@aiuto aiuto deleted the ntst branch February 15, 2023 17:06
@aiuto aiuto deleted the ntst branch February 15, 2023 17:06
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.

Provide a tool for testing content of zip/tar files
2 participants