Skip to content

Update .gitignore for ignoring goenv .go-version file - #2242

Merged
gmlewis merged 2 commits into
google:masterfrom
siansiansu:update-gitignore
Dec 27, 2021
Merged

Update .gitignore for ignoring goenv .go-version file#2242
gmlewis merged 2 commits into
google:masterfrom
siansiansu:update-gitignore

Conversation

@siansiansu

Copy link
Copy Markdown
Contributor

Hi reviewers:

I updated the .gitignore to ignore the .go-version file in this PR.
If merged, it's easy to manage different go version locally for the goenv user.
Please help to review it. Thanks!

Reference
https://github.com/syndbg/goenv/blob/master/COMMANDS.md#goenv-local

@siansiansu

Copy link
Copy Markdown
ContributorAuthor

I have signed the CLA, please help to re-run it if approved.

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @siansiansu !
Let's please make the comment a bit more useful.

Comment thread.gitignore Outdated
@codecov

codecovBot commented Dec 27, 2021

Copy link
Copy Markdown

Codecov Report

Merging #2242 (87a1844) into master (4d7a0b5) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #2242 +/- ##
=======================================
Coverage 97.80% 97.80% =======================================
Files 113 113 Lines 10205 10205 =======================================
Hits 9981 9981 Misses 156 156 Partials 68 68 

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d7a0b5...87a1844. Read the comment docs.

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @siansiansu !
LGTM.
Merging.

@gmlewis
gmlewis merged commit a899e0f into google:masterDec 27, 2021
@siansiansu
siansiansu deleted the update-gitignore branch December 28, 2021 02:05
jlaportebot added a commit to jlaportebot/go-github that referenced this pull request Jun 28, 2026
Sign up for freeto 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.

2 participants

@siansiansu@gmlewis