A RetroSearch Logo

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Search Query:

Showing content from https://github.com/pmd/pmd/issues/384 below:

[javascript] Trailing commas not detected on French default locale · Issue #384 · pmd/pmd · GitHub

Rule Set: basic.xml

Description: Hi! When trying to build pmd from source the build always fails because of the tests of AvoidTrailingComma in the javascript basic ruleset. I've investigated a bit and found that the rule invariably reports no violation. The rule is defined by the following xpath expression:

//ObjectLiteral[$allowObjectLiteral = "false" and @TrailingComma = 'true']
|
//ArrayLiteral[$allowArrayLiteral = "false" and @TrailingComma = 'true']

I've checked the AST of some test cases in the designer, and it appears the TrailingComma attribute of Object- and ArrayLiterals is set to false even though it should be true...

All of this is happening with a fresh clone of pmd... I think it might be a dependency problem because the same code passes the tests on travis and certainly your machines too. Any ideas ?

Code Sample demonstrating the issue:

x = {a : 1,};
x = {a : 1, b : 2,}; 

Building PMD through: mvn clean verify


RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4