-
Notifications
You must be signed in to change notification settings - Fork 14
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
Update to net472 and net.sdk #30
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
708bc3e
to
704f578
Compare
This comment has been minimized.
This comment has been minimized.
704f578
to
ae2cc6f
Compare
45ec6c3
to
bcbcc20
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
ok no problem, ive reference this TrimbleSolutionsCorporation/VSSonarQubeExtension#278 to upgrade the linter there to last version and i will do all the testing then in our internal servers also! |
9b2ccdd
to
7485cac
Compare
This comment has been minimized.
This comment has been minimized.
7485cac
to
32ee9a3
Compare
@jmecosta and everyone else: Any comments and help on open points are welcome. Especially on the two last ones, which are not so obvious. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Now with the updated pom files you can build locally everything with Maven again, with configuration Debug or Release:
|
This comment has been minimized.
This comment has been minimized.
@jmecosta Build 193-rc I installed on our server in production. If you could verify on your side that no crash occurs and some reports are published on Sonarqube Server I would like to squash merge this PR. On the next days I'll observe our server regarding F# builds. |
compilation now works fine thanks, im installed the plugin in server and run analysis all working perfectly fine. Thanks |
good to be merged |
# Conflicts: # FsSonarRunner/FsSonarRunner/AssemblyInfo.fs # FsSonarRunner/FsSonarRunnerCore.Test/AssemblyInfo.fs # FsSonarRunner/FsSonarRunnerCore/AssemblyInfo.fs # FsSonarRunner/pom.xml # appveyor.yml # pom.xml # sonar-fsharp-plugin/pom.xml
Update to net472 and net.sdk
Open points:
several links are broken in sonar-fsharp-plugin\src\main\resources\org\sonar\l10n\fsharp\rules\fsharplintIssue Broken links in locatization #49junit tests results are not reported in AppVeyor (only 21 F# Tests, 1 Junit test is missing)Issue Upload JUnit test results in CI build #50[fs-]analysis-(input|output).xml - which ones are needed, all required available?Issue Check which *analysis*.xml files are required #51pom files parent referencesfixed in PR Information in SonarQube-Administration-Marketplace #48Adjustments required to handle new rules ofFSharpLint (F# solution, Java wrapper plugin)Issue Update ruleset to match FsharpLint #64Test:
Reminder: DO SQUASH MERGE