Skip to content

fix: remove extra .tgz file #346

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 7, 2021
Merged

fix: remove extra .tgz file #346

merged 1 commit into from
Oct 7, 2021

Conversation

matthewrobertson
Copy link
Member

It looks like this file got added by mistake in a recent commit

@google-cla google-cla bot added the cla: yes label Oct 6, 2021
grant
grant approved these changes Oct 6, 2021
View reviewed changes
Copy link
Contributor

@grant grant left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Whoops. #328

Should we gitignore tgz / zip too?

It's from npm pack.

@matthewrobertson
Copy link
Member Author

@grant good idea. I will update the gitignore

@matthewrobertson
fix: remove tgz file
fab6685 
It looks like this file got added by mistake in a recent commit
@matthewrobertson matthewrobertson merged commit 51cb666 into master Oct 7, 2021
@grant grant deleted the remove-zip branch October 15, 2021 19:52
@grant grant restored the remove-zip branch October 15, 2021 19:52
@grant grant deleted the remove-zip branch November 18, 2021 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
2 more reviewers

@grant grant grant approved these changes

@anniefu anniefu anniefu approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@matthewrobertson @grant @anniefu