Skip to content

Conversation

@NTaylorMullen
Copy link

  • Includes a fix for the Razor server ready check.

- Includes a fix for the Razor server ready check.
@TanayParikh
Copy link
Contributor

@NTaylorMullen
Copy link
Author

Changelog needs to be updated with the new version: https://github.com/OmniSharp/omnisharp-vscode/blame/master/CHANGELOG.md#L18

Is this still true?
https://github.com/OmniSharp/omnisharp-vscode/blame/master/CHANGELOG.md#L6

Good eye! Will update

@codecov
Copy link

codecov bot commented Oct 30, 2020

Codecov Report

Merging #4166 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4166   +/-   ##
=======================================
  Coverage   85.99%   85.99%           
=======================================
  Files          60       60           
  Lines        1857     1857           
  Branches      215      215           
=======================================
  Hits         1597     1597           
  Misses        200      200           
  Partials       60       60           
Flag Coverage Δ
integration 100.00% <ø> (ø)
unit 85.99% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 a009182...29e1942. Read the comment docs.

@JoeRobich JoeRobich merged commit 3e4dd35 into master Oct 30, 2020
@JoeRobich
Copy link
Member

Rebuilt 1.23.5 available at https://github.com/OmniSharp/omnisharp-vscode/releases/tag/v1.23.5

@NTaylorMullen
Copy link
Author

Thank you Joey!

@NTaylorMullen NTaylorMullen deleted the nimullen/razorupdate.6.0.0-alpha.1.20529.17 branch November 2, 2020 18:22
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.

4 participants