Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parsing issue with default in export clause specifier #960

Closed
fxOne opened this issue Mar 20, 2018 · 5 comments
Closed

Parsing issue with default in export clause specifier #960

fxOne opened this issue Mar 20, 2018 · 5 comments
Assignees
Milestone

Comments

@fxOne
Copy link

fxOne commented Mar 20, 2018

I want to report a bug.

SonarJS version: 4.1 (build 6085)

SonarQube version: Version 6.7 (build 33306)

Rule key: javascript:ParsingError

Logs

15:35:13.752 ERROR - Unable to parse file: .../test.js
15:35:13.753 ERROR - Parse error at line 1 column 9:

1: export {default as template} from './otherFile.html';
            ^

Expected behavior
We updated today to the latest Version of SonarJS und after the scan we got multiple parse errors. In the previous it worked as expected - the files got parsed without errors. Unfortunately we don't know our previous version number.

@vilchik-elena
Copy link
Contributor

Probably it's due to Flow syntax support #584 (was added in 3.2)

@vilchik-elena vilchik-elena added this to the 4.2 milestone Mar 20, 2018
@hcomnetworkers
Copy link

Is there a way we can temporary tell Sonar to ignore theses exports until the bug is fixed? We have an export like this in hundreds of files, which are not getting analyzed since the update.

@vilchik-elena
Copy link
Contributor

@hcomnetworkers nope :(

@phillipuniverse
Copy link

For us, we are only using this syntax in our React index.js files so I excluded them with sonar.javascript.exclusions=**/index.js.

@cambiph
Copy link

cambiph commented Jul 4, 2018

Still a problem on SonarQube v7.2 (build 13530) and SonarJS 4.1 (build 6085).

@saberduck saberduck self-assigned this Jul 11, 2018
@ghost ghost removed the review label Jul 11, 2018
@nicolas-harraudeau-sonarsource nicolas-harraudeau-sonarsource changed the title javascript:ParsingError with export Fix parsing issue with default in export clause specifier Jul 23, 2018
@nicolas-harraudeau-sonarsource nicolas-harraudeau-sonarsource changed the title Fix parsing issue with default in export clause specifier Parsing issue with default in export clause specifier Jul 23, 2018
ksokol pushed a commit to ksokol/myreader that referenced this issue Feb 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants