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

feat: tar includes runfiles #595

Merged
merged 7 commits into from
Oct 9, 2023
Merged

feat: tar includes runfiles #595

merged 7 commits into from
Oct 9, 2023

Conversation

alexeagle
Copy link
Collaborator

@alexeagle alexeagle commented Oct 7, 2023

Type of change

  • New feature or functionality (change which adds functionality)

For changes visible to end-users

  • Relevant documentation has been updated

Test plan

  • New test cases added

@alexeagle alexeagle force-pushed the tar_runfiles branch 7 times, most recently from 1cb96cd to 8d6238b Compare October 8, 2023 23:13
@alexeagle alexeagle force-pushed the tar_runfiles branch 2 times, most recently from f9e6569 to 879c082 Compare October 9, 2023 21:25
@alexeagle alexeagle force-pushed the tar_runfiles branch 2 times, most recently from 9c8190b to 8fc6e10 Compare October 9, 2023 22:01
@alexeagle alexeagle merged commit 472bf9b into 2.x Oct 9, 2023
35 checks passed
@alexeagle alexeagle deleted the tar_runfiles branch October 9, 2023 22:57
alexeagle added a commit that referenced this pull request Dec 23, 2023
* feat: tar includes runfiles

* chore: try to fix red circleci

* fix: tracked down problem

* chore: document tar#srcs supports runfiles

* chore: add comment about logic for trimming manifest suffix

* chore: missed a replacement spot

* chore: give up on the listing test for now
gregmagolan pushed a commit that referenced this pull request Mar 23, 2024
* feat: tar includes runfiles

* chore: try to fix red circleci

* fix: tracked down problem

* chore: document tar#srcs supports runfiles

* chore: add comment about logic for trimming manifest suffix

* chore: missed a replacement spot

* chore: give up on the listing test for now
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.

3 participants