-
Notifications
You must be signed in to change notification settings - Fork 0
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
merge from pmd/pmd #1
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[core] Update asm and openfjx
[plsql] Support for SELECT INTO
[plsql] Support update statement
[plsql] Support Function Calls in Select
The current CPD documentation covers very well how to use it, but there was no information here that describes why someone would care about how much duplicate code they have. The developers I work with want to "cheat" on their SonarQube quality gates by excluding some code from CPD checks, because they don't really understand what the problem is. Ironically, the main risk from duplicate code is when that code unintentionally becomes not duplicate anymore, creating a hidden problem that may or may not erupt on you. The text I've added here is excerpted from a short tech note I wrote to my team on the subject.
[plsql] Support Delete Statement
Note: ALL and DISTINCT are reserved words
Note: BY and GROUP are reserved words
Increase lookaheads in Condition2
Special handling of TableAlias in Joins ON is a reserved word
TRAVIS_JOB_NUMBER=3007.1 TRAVIS_COMMIT_RANGE=39d340f431c1...1d9a412
This reverts commit 2ccfe15.
* This makes it easier to create a customized and tailored PMD distribution. * Also sort the languages alphabetically
TRAVIS_JOB_NUMBER=3044.1 TRAVIS_COMMIT_RANGE=2aae4b0272e0...b0518c9
[ui] make the language dependencies optional
[docs] Update "Testing your rules"
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please, prefix the PR title with the language it applies to within brackets, such as [java] or [apex]. If not specific to a language, you can use [core]
Before submitting a PR, please check that:
master
. The PMD team will merge back to support branches as needed.mvn test
passes.mvn checkstyle:check
passes. Check this for more infoPR Description: