Comments (17)
Hi @Daniel1989 ,
thanks for bringing this up, currently we are considering to parse the javascript code inside the .vue <script>
tag so that we could analyse it like any other js code sitting in a .js file. What do you think?
from sonarjs.
@kamontat yes, it is supported. This debug message you are seeing is coming from the new parser, but it's not relevant for the analysis.
from sonarjs.
while this is in progress, I think we can use this guide as workaround,
https://vuejs.org/v2/guide/single-file-components.html#What-About-Separation-of-Concerns
externalize the js file from the vue file.
from sonarjs.
It is great for us too
from sonarjs.
I just tested with latest version v6.5 sonarqube on my local, it works as expected. it seems the LTS version we used with CI was using an old version of SonarJs. Sorry for the confusion and thank you for help.
from sonarjs.
Hi @inverno ,
Thanks for your reply, I think your idea is great for me. Could I know when this feature will be available?
from sonarjs.
@Daniel1989
We are considering this feature for next iteration. As soon as it enters an iteration you'll see it here.
Pull requests, of course, are always welcome :-)
from sonarjs.
@inverno Great thanks!
from sonarjs.
Reopening this. We'll close it when implemented :)
from sonarjs.
WIll be better for support ES6 syntax(ts as next feature).
A lot of projects used *.vue extension. Thanks
from sonarjs.
- add some vue files to Ruling IT
from sonarjs.
@vilchik-elena , any update for the feature availability?
from sonarjs.
@pwang2 it's been completed and released in SonarJS 3.1, see the milestone : https://github.com/SonarSource/sonar-javascript/milestone/4?closed=1
from sonarjs.
From the link in the repo description, I only see jsx support not vue. I have sonarjs 3.1.1 installed and even after setup suffixes, still is in my vue file is ingored. Did I miss anything here? Thanks!
from sonarjs.
@pwang2 indeed we did not add .vue
to the supported frameworks list, but that's just text. The .vue
suffix is pre-configured in 3.1.1 and I just analyzed https://github.com/vuejs-templates/webpack-simple (which contains a .vue
file) without any extra configuration.
Mind to provide the log of the analysis, the configuration you are using and maybe a sample .vue
file that is failing to be analyzed?
from sonarjs.
@pwang2 no problem, good to know that it works for you
from sonarjs.
Is it support VueJS? Why is still have skip Vue files ??
Skipping analysis of Vue.js file <path>/default.vue
from sonarjs.
Related Issues (20)
- update all (easy) dependencies and evaluate upgrading hard ones
- Create artifact which embeds Node.JS for Alpine Linux HOT 1
- FP S6827: Thymeleaf `th:text` attribute is not recognized as a content HOT 1
- Do not count LOCs for CSS files indexed as test files
- Exclude assignment expressions to `require()` from copy-paste detection HOT 1
- javascript/S6654.html is missing `Of` in references to prototype getter/setter methods of Object
- Fix FP S6754 (`hook-use-state`): Ignore state variables without a setter
- Fix FP S2699 (`assertions-in-tests`): Add support for `supertest` HOT 1
- Fix FP S6747 (`no-unknown-property`): Don't raise for `css` attribute HOT 1
- Fix FP S6747 (`no-unknown-property`): Don't raise for allowed attributes HOT 1
- Fix FP S6747 (`no-unknown-property`): Don't raise issues from `eslint-plugin-react/no-unknown-property` when react is not used HOT 1
- Embedded Node.js runtime: detect the host architecture correctly in MacOS JRE Intel virtualization HOT 1
- Fix FP S6754 (`hook-use-state`): Add exception when using `useState` for components HOT 1
- Embedded Node.js: when `sonar.nodejs.forceHost` is set, do not deploy the runtime HOT 1
- Remove usage of deprecated logging APIs
- JS-18 S6819 False Positive when using SAP UI5 React components HOT 1
- Use 18.17.0 as minimum version for Node.JS
- Ignore runtime errors when saving cpd tokens
- Incorrect S6757 Error for Class Components HOT 8
- We should skip the whole file when we fail to save CPD token
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 sonarjs.