Skip to content

Conversation

@korableg
Copy link

Hello!
I created my own linter which checks proto generated files (I need checking options) and faced with the same issue as here #2864 that I can't to check the generated proto files.

This pull request fixes this problem by adding new config parameter.

Check plz :) Thank you!

@boring-cyborg
Copy link

boring-cyborg bot commented May 18, 2024

Hey, thank you for opening your first Pull Request !

@CLAassistant
Copy link

CLAassistant commented May 18, 2024

CLA assistant check
All committers have signed the CLA.

@ldez ldez self-requested a review May 18, 2024 15:18
@ldez ldez added the blocked Need's direct action from maintainer label May 18, 2024
@ldez ldez changed the title add include generated include generated files May 18, 2024
@ldez ldez added declined and removed blocked Need's direct action from maintainer labels May 21, 2024
@ldez
Copy link
Member

ldez commented May 21, 2024

Hello,

Your first was better than the current one.

The current implementation is a kind of weird opposite of exclude-dirs and exclude-files mixed with the management of the generated files.

This implementation is not in the right direction.

Supersedes by #4740

@ldez ldez closed this May 21, 2024
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.

3 participants