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.

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

Merged
merged 62 commits into from
Jun 30, 2019

Conversation

milbrandt
Copy link
Collaborator

@milbrandt milbrandt commented May 30, 2019

  • Updated Project files to new Project Format style
  • Updated from .NET Framework 4.5 to 4.7.2 (not going to dotnet Core as FSharp Compiler.Service.* is not available therefore)
  • added NUnit TestAdapter for Visual Studio 2017 as Nuget package
  • updated most Nuget packages (all except for FSharp Compiler.Service.* and FSharpLint.Core)

Open points:

Test:

Reminder: DO SQUASH MERGE

@milbrandt milbrandt mentioned this pull request May 30, 2019
@jmecosta

This comment has been minimized.

@jmecosta

This comment has been minimized.

@milbrandt

This comment has been minimized.

@jmecosta

This comment has been minimized.

@jmecosta

This comment has been minimized.

@milbrandt milbrandt force-pushed the netSdkProjectFormat branch from 704f578 to ae2cc6f Compare May 31, 2019 17:20
@milbrandt milbrandt force-pushed the netSdkProjectFormat branch 2 times, most recently from 45ec6c3 to bcbcc20 Compare June 2, 2019 20:26
@milbrandt

This comment has been minimized.

@jmecosta

This comment has been minimized.

@milbrandt

This comment has been minimized.

@milbrandt milbrandt self-assigned this Jun 8, 2019
@jmecosta

This comment has been minimized.

@milbrandt

This comment has been minimized.

@jmecosta

This comment has been minimized.

@jmecosta
Copy link
Owner

jmecosta commented Jun 8, 2019

@jmecosta Unfortunately not yet :( There is still an issue with the plugin build in the pom. ANd then we have to test - it is update with lots of breaking changes in the dependencies.

But on the bright side: the number of rules have increased in FsharpLint and I could create some additional PRs on other projects

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!

@milbrandt milbrandt force-pushed the netSdkProjectFormat branch from 9b2ccdd to 7485cac Compare June 8, 2019 20:18
@milbrandt

This comment has been minimized.

@milbrandt milbrandt force-pushed the netSdkProjectFormat branch from 7485cac to 32ee9a3 Compare June 12, 2019 21:40
@milbrandt
Copy link
Collaborator Author

@jmecosta and everyone else: Any comments and help on open points are welcome. Especially on the two last ones, which are not so obvious.

@jmecosta

This comment has been minimized.

@jmecosta

This comment has been minimized.

@milbrandt

This comment has been minimized.

@milbrandt

This comment has been minimized.

@milbrandt

This comment has been minimized.

@milbrandt
Copy link
Collaborator Author

Looks like there is no more run in pom.xml to build the solution

? so how do I build the all thing locally without manually building the core first

Now with the updated pom files you can build locally everything with Maven again, with configuration Debug or Release:

mvn -Dconfiguration=Debug clean package

@milbrandt

This comment has been minimized.

@milbrandt
Copy link
Collaborator Author

@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.

@jmecosta
Copy link
Owner

compilation now works fine thanks, im installed the plugin in server and run analysis all working perfectly fine. Thanks

@jmecosta
Copy link
Owner

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
@milbrandt milbrandt changed the title [Test] Update to net472 and net.sdk Update to net472 and net.sdk Jun 30, 2019
@milbrandt milbrandt merged commit 54546e6 into jmecosta:master Jun 30, 2019
milbrandt added a commit to milbrandt/sonar-fsharp-plugin that referenced this pull request Jun 30, 2019
@milbrandt milbrandt deleted the netSdkProjectFormat branch July 1, 2019 18:33
@milbrandt milbrandt mentioned this pull request Jul 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants