Comments (3)
Hi @Bertk,
Thanks for your feedback!
Pystrings cannot be ignored because they can contain placeholders.
I'll try to remove those false positives by checking if they are preceded by the include keyword.
Tell me what you think.
Thank you
Have a good day!
David
from sonar-gherkin-plugin.
Hi David,
I am not sure whether adding a check for #include pre-processor directive is necessary.
This feature file is used for the C++ Community plugin tests and nobody else will do this.
Using escape sequences "\<", "\>" might be a workaround. If this is successful a hint in the issues to use escape sequence would be nice.
I will check this with a pull request tomorrow.
Best regards
Bert
https://github.com/cucumber/cucumber/blob/master/gherkin/docs/ast.pdf
from sonar-gherkin-plugin.
Hi David,
I checked with this PR SonarOpenCommunity/sonar-cxx#1609 and the escape sequence did not break the test.
I am still not sure whether a Pystring can be used to define a symbolic name for values.
Regards
Bert
from sonar-gherkin-plugin.
Related Issues (20)
- New rule: Features should be written in the same language
- Support ``` as DocString prefix/suffix
- please provide stand alone maven example to use it HOT 4
- Feature files not picked up HOT 3
- Support scenarios with two 'Examples' sections HOT 2
- False positive when a variable is used inside a table HOT 1
- Change 'allowed-tags' rule parameter from a comma-separated list to a regular expression HOT 2
- Metrics in SonarQube 6.6 HOT 2
- Make the plugin compatible with SonarQube 6.7
- FP on rule unused-variable: Variable in doc string not recognized HOT 3
- Check that .feature files exist in the project HOT 4
- Plugin not working when used with SonarCloud HOT 4
- add sonarlint supported config prop HOT 2
- False positive on Common Given steps should be added to Background HOT 2
- Cucumber Gherkin plugin fails with JAXBException (SQ 7.9.1, JRE 11) HOT 2
- Error during SonarQube Scanner execution : java.lang.NoClassDefFoundError: javax/xml/bind/JAXBException HOT 1
- New Release HOT 2
- No issues raised against gherkin cutom rules HOT 1
- Update the project 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 sonar-gherkin-plugin.