use SQ API 6.7 #1291
Merged
use SQ API 6.7 #1291
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
|
@Bertk like to release 0.9.8 first. Will shift review after release... |
|
OK. The CI checks are failing and I disabled feature "Importing Cppcheck ANSI-C reports" . I declared the header and source file extension as multivalue and the syntax of the request changed. |
Bertk
force-pushed
the
SQAPI6.5
branch
2 times, most recently
from
29da60c
to
19c9b07
Compare
- 2 coverage unit tests are failing (now disabled) - sslr-cxx-toolkit build fails (INVOKESPECIAL/STATIC interfaces require ASM 5) - integration tests not checked fix coverage unit tests update version to 0.9.9 fix optinal and unit tests
- dotnet.tests.UnitTestResultsAggregator not compatible with org.sonar.api.config.Configuration
- replace org.fest.assertions.Assertions with org.assertj.core.api - remove sonar-dotnet-tests-library
avoid warnings and add log messages Fix messages like: WARNING: WARN: Property 'sonar.cxx.vera.reportPath' is not declared as multi-values/property set but was read using 'getStringArray' method. The SonarQube plugin declaring this property should be updated. cleanup of quality flaws remove warnings and error messages
reduce size of CXX sslr toolkit - remove includes - fix some issues
fix unit tests and clean-up fix typo more version id updates use SQ 6.7 API and use list of files for coverage
drop details for exception - disable failing API call enable importing_cppcheck_reports_c_cpp test fix python for loop use dictionary correct way
- check later
setting file extension does not work 11:55:23.348 ERROR: Error during SonarQube Scanner execution Language of file 'src\component1.h' can not be decided as the file matches patterns of both sonar.lang.patterns.c++ : **/*.cpp,**/*.cc,**/*.hxx,**/*.hpp,**/*.hh,**/*.h and sonar.lang.patterns.c : **/*.c,**/*.h
|
@ALL someone else wanna review it? |
|
@guwirth Cleanup is still not completed and will be addressed with a another PR. |
This was referenced Nov 20, 2017
This was referenced Nov 20, 2017
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.
#1272
This change is