Comments (4)
@Kresshy Sorry for the late response. At the moment, this feature is not supported.
I acknowledge that this feature could be worthwhile.
from protolint.
+1 for this feature, would be very useful to exclude linting on proto files imported from external repositories that we don't have/want control over.
from protolint.
@yoheimuta Thanks for this nice utility. I am interested in contributing this feature.
Do you prefer pattern matching via glob as suggested in this issue, or regex? In my opinion globs are nicer to work with, but the standard library implementation in filepath doesn't support the ** pattern which many users will expect to be supported. There are third-party libraries such as https://github.com/bmatcuk/doublestar that could be used instead.
For backwards-compatibility reasons I think we'd have to introduce a new field (exclude_pattern ?) for both directories and files.
from protolint.
@stevenschaerer Thanks for the suggestion!
I can see the appeal of globs for some users, so adding support for them seems like a great idea.
I also like the idea of introducing a new field.
from protolint.
Related Issues (20)
- [Reporters] Add ability to use multiple reporters HOT 4
- [Distribution] Add support for C#/VB/F# (.NET Toolchain) HOT 2
- [Distribution] Add support for Python HOT 3
- [Distribution] Add support for Typescript / Javascript
- FILE_NAMES_LOWER_SNAKE_CASE: an issue with dot separated filenames HOT 2
- Bug: Enum Field Prefix check HOT 3
- Minor problem disabling warnings. HOT 3
- `INDENT` rule: incorrectly works with multiline comments HOT 1
- Bug: Enum field prefix naming rule HOT 3
- Stray double semi-colon cause erratic results
- protolint npm package isn't available HOT 3
- Error thrown when installing protolint through yarn HOT 1
- Can't read Unix domain socket as a file
- FILE_NAMES_LOWER_SNAKE_CASE fails to convert kebab case to snake case HOT 2
- RPC_NAMES_UPPER_CAMEL_CASE disable is ignored HOT 1
- Some rules do not support auto-disable HOT 5
- Allow exclusion list of field names for `FIELDS_HAVE_COMMENT` HOT 1
- ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH failing even with `allow_alias = true` HOT 1
- CVEs in the latest version HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from protolint.