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

txtar: add fs.FS support #285

Closed
wants to merge 1 commit into from
Closed

Conversation

earthboundkid
Copy link
Contributor

@earthboundkid earthboundkid commented Mar 13, 2021

Fixes golang/go#44158

From implementation copied with permission from https://github.com/josharian/txtarfs/ golang/go#44158 (comment)

Co-authored-by: Josh Bleecher Snyder josharian@gmail.com

@google-cla
Copy link

google-cla bot commented Mar 13, 2021

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to determine that you authored the commits in this PR. Maybe you used a different email address in the git commits than was used to sign the CLA? If someone else authored these commits, then please add them to this pull request and have them confirm that they're okay with them being contributed to Google. If there are co-authors, make sure they're formatted properly.

In order to pass this check, please resolve this problem and then comment@googlebot I fixed it... If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no label Mar 13, 2021
@earthboundkid
Copy link
Contributor Author

@googlebot I fixed it.

@google-cla
Copy link

google-cla bot commented Mar 13, 2021

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to determine that you authored the commits in this PR. Maybe you used a different email address in the git commits than was used to sign the CLA? If someone else authored these commits, then please add them to this pull request and have them confirm that they're okay with them being contributed to Google. If there are co-authors, make sure they're formatted properly.

In order to pass this check, please resolve this problem and then comment@googlebot I fixed it... If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@josharian
Copy link
Contributor

josharian commented Mar 13, 2021

I think if you s/@josharian/Josh Bleecher Snyder <josharian@gmail.com>/ in the commit message it should placate the CLA bot.

Fixes #44158

From implementation copied with permission from
https://github.com/josharian/txtarfs/blob/main/txtarfs_test.go
golang/go#44158 (comment)

Co-authored-by: Josh Bleecher Snyder <josharian@gmail.com>
@google-cla
Copy link

google-cla bot commented Mar 13, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@josharian
Copy link
Contributor

@googlebot I consent.

@google-cla
Copy link

google-cla bot commented Mar 13, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@josharian
Copy link
Contributor

Oh well. Listing me as a co-author was a nice thought. :P

@earthboundkid
Copy link
Contributor Author

@googlebot I consent.

@google-cla
Copy link

google-cla bot commented Mar 13, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@earthboundkid
Copy link
Contributor Author

I'm going to try closing this PR and opening a new one.

@earthboundkid earthboundkid deleted the add-fs branch March 15, 2021 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

x/tools/txtar: implement fs.FS
2 participants