Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Adds support for Apex classes (Salesforce)#29

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
filipesperandio merged 1 commit intocodeclimate:masterfromkevinohara80:master
Nov 7, 2018

Conversation

kevinohara80
Copy link
Contributor

Currently, Salesforce developers cannot use CodeClimate despite CodeClimate having PMD as a plugin. This is because the PMD plugin filters out all files that are not Java (.java). This patch allows for the Apex class extension to be processed as well (.cls).

Related to#26

@oded-rosi-samanage
Copy link

oded-rosi-samanage commentedNov 29, 2018
edited
Loading

Hey@kevinohara80 , Unfortunately we were using the pmd module in CodeClimate and it stopped working after this merge. right now I can read the issue is that the .cls file is missing from the project, although they exist in the repo (just referenced to another place in the project).
engine

After some reading i got to the ApexMetrics module, which should be discontinued shortly
It seemed to be working for over 6 months (at least) on the apex classes in git, without using the ApexMetrics module (I wasn't even aware of it).

Please advice

@filipesperandio
Copy link
Contributor

Hey@oded-rosi-samanage, if I understood the problem correctly, I believe you could fix the problem by adding anexclude_pattern configuration to you.codeclimate.yml config.
You can add any kind of exclusion for an specific plugin, it could be all.cls files, or specific ones (for example, files causing problems by referring to others not reachable by the analysis):

plugins:pmd:enabled:trueexclude_patterns:      -"**/*.cls"

@oded-rosi-samanage
Copy link

Thanks for the quick answer@filipesperandio , it does work as expected

filipesperandio reacted with thumbs up emoji

KroArtem pushed a commit to KroArtem/codeclimate-pmd that referenced this pull requestJan 13, 2021
Currently, Salesforce developers cannot use CodeClimate despite CodeClimate having PMD as a plugin. This is because the PMD plugin filters out all files that are not Java (.java). This patch allows for the Apex class extension to be processed as well (.cls).Related tocodeclimate#26
KroArtem pushed a commit to KroArtem/codeclimate-pmd that referenced this pull requestJan 13, 2021
Currently, Salesforce developers cannot use CodeClimate despite CodeClimate having PMD as a plugin. This is because the PMD plugin filters out all files that are not Java (.java). This patch allows for the Apex class extension to be processed as well (.cls).Related tocodeclimate#26
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees

@filipesperandiofilipesperandio

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@kevinohara80@oded-rosi-samanage@filipesperandio

[8]ページ先頭

©2009-2025 Movatter.jp