Generated files with copyright notices should be skipped, too

[AlekSi]

go-consistent/main.go

Lines 192 to 193 in c48b49e

	isGenerated := len(f.Comments) != 0 &&
	generatedFileCommentRE.MatchString(f.Comments[0].Text())

I think that logic does not properly ignore files with copyright notices like:

// Copyright 2026 Google LLC
//
// Licensed under bla-bla-bla.

// Code generated by protoc-gen-go. DO NOT EDIT.
// versions:
// 	protoc-gen-go v1.36.11
// 	protoc        (unknown)
// source: google/type/decimal.proto

[cristaloleg]

Why do you scan dependencies? or vendor at all.

[AlekSi]

In our case, it is neither a dependency nor a vendor directory. That could be any file that has any comment block before the DO NOT EDIT comment. The copyright notice is quite common.

[cristaloleg]

Ok, just a bad example.

[AlekSi]

I'm not sure why that is a bad example. That's a real file. Not a dependency. No in the vendor directory. A real generated file in our code. Yes, we do have a copy of https://github.com/googleapis/googleapis/blob/master/google/type/decimal.proto in our repo, but we generate decimal.pb.go ourselves.