Skip to content

Conversation

@JoeRobich
Copy link
Member

Update to match the OmniSharp-Roslyn required Mono version.
@JoeRobich
Copy link
Member Author

@filipw Would there be any reason to allow the use of a earlier version?

@filipw
Copy link
Contributor

filipw commented Jan 30, 2020

no makes sense to me 👌

@filipw
Copy link
Contributor

filipw commented Jan 30, 2020

the server allows 6.4.0 by the way, when that change was made 6.6.0 was still in pre-release
https://github.com/OmniSharp/omnisharp-roslyn/blob/master/src/OmniSharp.Host/MSBuild/Discovery/Providers/MonoInstanceProvider.cs#L72

but it should be aligned at 6.6.0 since embedded mono is 6.6.0 based

@codecov
Copy link

codecov bot commented Jan 30, 2020

Codecov Report

Merging #3539 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #3539   +/-   ##
======================================
  Coverage    89.8%   89.8%           
======================================
  Files          59      59           
  Lines        1589    1589           
  Branches       89      89           
======================================
  Hits         1427    1427           
  Misses        151     151           
  Partials       11      11
Flag Coverage Δ
#integration 100% <ø> (ø) ⬆️
#unit 89.8% <100%> (ø) ⬆️
Impacted Files Coverage Δ
src/omnisharp/OmniSharpMonoResolver.ts 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6b97ec9...ba98810. Read the comment docs.

@JoeRobich JoeRobich merged commit 4acb01d into master Jan 30, 2020
@JoeRobich JoeRobich deleted the dev/jorobich/update-required-mono branch February 26, 2020 23:13
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.

3 participants