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.

Sign up for GitHub

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

Jump to bottom

Feature/checkstyle renderer #1

Open
wants to merge 145 commits into
base: master
Choose a base branch
from
Open

Feature/checkstyle renderer #1

wants to merge 145 commits into from

Conversation

ffflabs
Copy link
Owner

@ffflabs ffflabs commented Jan 4, 2021

Demonstration of automated PR annotations using cs2pr with phpmd's checkstyle renderer

tvbeek and others added 30 commits May 7, 2020 22:04
@tvbeek
Rename setIgnorePattern to addIgnorePatterns and getIgnorePattern to …
d383bd3 
…getIgnorePatterns so they are less confusion.
@tvbeek
Update the use of the deprecated functions from PHPMD in the ParserFa…
4f1586c 
…ctory.
@tvbeek
Update docblocks with feedback from @ravage84
d801efd
@tvbeek
Fix the mocking in the ParserFactoryTest after using the new functions
7ec1d8d
@tvbeek
Remove unwanted empty lines
754b06b
@tvbeek
Update the docblock with @this and add an empty line. Feedback from @…
0bed148 
…kylekatarnls
@kylekatarnls
Fix phpmd#673 Ignore dynamic class name from missing imports
554a474
@kylekatarnls
Migrate MissingImport to the new testing system
f6b6f1b
@kylekatarnls
phpmd#739 Add unit test to demonstrate the class begin line issue
aa55524
@kylekatarnls
Merge branch 'master' into feature/issue-673-ignore-dynamic-class-names
3ba954e
@kylekatarnls
Cleanup merge changes
b125659
@kylekatarnls
phpmd#739 Upgrade minimum version of PDepend
fc816ad
@kylekatarnls
phpmd#739 Shorten variable name numberOfPublicMethods > publicMethods…
0b1d9b6 
…Count
@kylekatarnls
phpmd#818 Don't check function arguments if the callee is a method
2e0b64e
@kylekatarnls
Fix phpmd#826 consider foreach exception only for direct children
0f224c4
@tvbeek
Change the naming error where possible
3429c96
@babanesma
recognize compact variables with double quotes
f63cbdc
@kylekatarnls
Merge branch 'master' of https://github.com/phpmd/phpmd into feature/…
5f25f6c 
…issue-818-unit-tests
@tvbeek
Remove the incorrect robots.txt and the reference in the site.ini
0d1d21d
@SanderSander
Fix multiple underscores in method name when allow-underscore-test is…
429dd39 
… enabled in CamelCaseMethodName
@kylekatarnls
Simplify RegExp
3fdd5b2
@SanderSander
Improve test documentation
953417c
@ffflabs
adds checkstyle compatible renderer, suitable for cs2pr or reviewdog
e0ff386
@ffflabs
repairs PSR2 code style faults, adds description to README
4644053
@lukasbestle
Add GitHub renderer for GitHub Actions
17558c7
@kylekatarnls @MarkVaughn
Cleanup
ce97f2d 
Co-authored-by: Mark Vaughn <[email protected]>
@ffflabs @kylekatarnls
Update src/main/php/PHPMD/TextUI/CommandLineOptions.php
ae0fbb4 
Co-authored-by: Kyle <[email protected]>

Update src/test/php/PHPMD/TextUI/CommandTest.php

Co-authored-by: Kyle <[email protected]>
@ffflabs
Inherit from XMLRenderer where it's due
5e8d195 
  - Also: declare a method showing equivalence between priority and
severity
@ffflabs
Use method 'maybeAdd' from parent renderer
7027f9f
@ffflabs
Remove scalar type declarations and keep file path
9e4ab02 
  - As phpmd is meant to run in a broad range of php versions,
    we cannot rely on features that wouldn't run in PHP 5.4.x
  - Violation fileName shoulnd't be modified. Any path canonicalization
    needed by an external tool should be applied afterwards
kylekatarnls and others added 30 commits April 11, 2021 16:53
@kylekatarnls
Merge pull request phpmd#873 from frankdekker/violation-baseline
281a8da 
Added ability to baseline existing violations
@frankdekker
Merge pull request #1 from phpmd/master
1027dee 
update master
@frankdekker
Fix slashes when running tests on windows environment
42a2686
@kylekatarnls
Merge pull request phpmd#874 from TJVB/update_contributing
23f62fa 
Update CONTRIBUTING.md
@kylekatarnls
Merge pull request phpmd#875 from frankdekker/violation-baseline-appv…
95efe0b 
…eyor-fixes

Violation baseline appveyor fixes
@frankdekker
Add BaselineMode + Update CommandLineOptions to support --update-base…
e2241d9 
…line
@frankdekker
Replace BaselineSet with Report in PHPMD::processFiles method
16dcb73
@frankdekker
Added BaselineValidator
c14a9fe
@frankdekker
Add tests for BaselineValidator
9056d48
@frankdekker
Add tests for BaselineValidator
6fa9c52
@frankdekker
Add tests for BaselineValidator
09b1610
@frankdekker
Fixes: unlink($tempFile): Resource temporarily unavailable
f1eaecf
@frankdekker
Merge pull request #2 from phpmd/master
0987138 
Merge phpmd master
@frankdekker
Add tests for BaselineValidator
cf1be81
@frankdekker
Remove unused imports
1aba7ac
@frankdekker
Add coverage Command
060e648
@frankdekker
Add coverage CommandTest for update baseline
7b9cc40
@frankdekker
Update documentation
7677002
@frankdekker
Update documentation
4363588
@frankdekker
Update documentation
3861911
@frankdekker
Update documentation
5ee4218
@frankdekker
Update documentation
dacb391
@frankdekker
Appease scrutinizer: suppress PHP7.4 getMock error
07dd69d
@tvbeek
Merge pull request phpmd#876 from frankdekker/update-violation-baseline
902a340 
Added option --update-baseline to update an existing phpmd.baseline.xml
@kylekatarnls
Merge branch 'master' into feature/sarif-renderer
064810c
@kylekatarnls
Indent long line
7ffb7b7
@kylekatarnls
Merge pull request phpmd#865 from lukasbestle/feature/sarif-renderer
c826a7c 
Add SARIF renderer
@kylekatarnls
Merge branch 'master' into feature/checkstyle_renderer
b5421a0
@kylekatarnls
Stick cast to variable (PHPCS compliance)
fdd2e37
@kylekatarnls
Align PHPDoc asterix
0101e01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@ffflabs @tvbeek @kylekatarnls @babanesma @SanderSander @lukasbestle @danepowell @frankdekker