Skip to content
This repository was archived by the owner on Apr 1, 2020. It is now read-only.

Feature/added csharp textmate grammar #2539

Merged

Conversation

MattMcLennan
Copy link
Contributor

Added text mate grammar for C#

@MattMcLennan
Copy link
Contributor Author

MattMcLennan commented Sep 1, 2018

According to AppVeyor they have increased the allowed artifact storage. Can we get the AppVeyor to be rurun for this pull request @Akin909

https://help.appveyor.com/discussions/problems/16350-oni-maximum-allowed-artifact-storage-size-of-70000-mb-will-be-exceeded

Copy link
Member

@CrossR CrossR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good, thanks for the contribution!

Just need to wait for the CI, then can be merged.

@codecov
Copy link

codecov bot commented Sep 1, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2539   +/-   ##
=======================================
  Coverage   44.76%   44.76%           
=======================================
  Files         351      351           
  Lines       14301    14301           
  Branches     1865     1865           
=======================================
  Hits         6402     6402           
  Misses       7673     7673           
  Partials      226      226
Impacted Files Coverage Δ
...src/Services/Configuration/DefaultConfiguration.ts 87.5% <ø> (ø) ⬆️

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 4bdbdc5...6ac1894. Read the comment docs.

@akinsho akinsho merged commit 98d129e into onivim:master Sep 1, 2018
@MattMcLennan MattMcLennan deleted the feature/added-csharp-textmate-grammar branch September 2, 2018 04:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants