search

onivim / oni

Oni: Modern Modal Editing - powered by Neovim
https://www.onivim.io
MIT License
11.35k stars 299 forks source link

Appveyor: remove artifact publish on master #2603

Closed bryphe closed 6 years ago

bryphe commented 6 years ago

We're hitting an appveyor error:

Uploading artifacts...
Maximum allowed artifact storage size of 90000 Mb will be exceeded

We actually only need the appveyor artifacts for releasing builds - so I've removed it so we don't hit this limit (and keep getting build errors).

These are still released on our staging and stable branches, so if we release from there, we'll still get those artifacts we need for release. Also, we'll still always upload to the azure storage for master builds.

codecov[bot] commented 6 years ago

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2603   +/-   ##
=======================================
  Coverage   45.26%   45.26%           
=======================================
  Files         361      361           
  Lines       14564    14564           
  Branches     1908     1908           
=======================================
  Hits         6593     6593           
  Misses       7747     7747           
  Partials      224      224

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 5c9a9cd...a4afd5b. Read the comment docs.

bryphe commented 6 years ago

I'm also following up with AppVeyor support, to see if we can clean out our existing artifacts to unblock PR / release: https://help.appveyor.com/discussions/problems/17235-hitting-upload-limit-for-builds