Skip to content

Skip generated files#12

Merged
jcchavezs merged 2 commits intojcchavezs:mainfrom
mx-psi:mx-psi/exclude-autogenerated
Oct 10, 2021
Merged

Skip generated files#12
jcchavezs merged 2 commits intojcchavezs:mainfrom
mx-psi:mx-psi/exclude-autogenerated

Conversation

@mx-psi
Copy link
Copy Markdown
Contributor

@mx-psi mx-psi commented Oct 8, 2021

Skip generated files based on convention from https://golang.org/s/generatedcode

Comment thread import.go Outdated
Comment thread import.go
var (
errMainPackage = errors.New("failed to add import to a main package")
errGenerated = errors.New("failed to add import to a generated file")
// Matches https://golang.org/s/generatedcode and cgo generated comment.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

<3 love the comments

@jcchavezs
Copy link
Copy Markdown
Owner

Just minor nit. Otherwise it looks great!

@mx-psi mx-psi requested a review from jcchavezs October 8, 2021 15:00
@jcchavezs jcchavezs merged commit 752edbc into jcchavezs:main Oct 10, 2021
@jcchavezs
Copy link
Copy Markdown
Owner

Thanks @mx-psi!

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.

2 participants