8000 Replace failing CoverityPublisher with plain 7zip and Curl by nulltoken · Pull Request #1078 · libgit2/libgit2sharp · GitHub
[go: up one dir, main page]
Skip to content

Replace failing CoverityPublisher with plain 7zip and Curl #1078

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 8, 2015
Merged

Replace failing CoverityPublisher with plain 7zip and Curl #1078

merged 1 commit into from
Jun 8, 2015

Conversation

nulltoken
Copy link
Member

Sadly CoverityPublisher fails. This PR fixes this by throwing curl in the game.

Current build behavior is now:

  • Coverity analysis only happens on scheduled builds
  • Tests aren't run on scheduled builds
  • NuGet package generation only happens on merge commits
  • Coverity/NuGget publishing only happens on the last job of the matrix

Curious minds can investigate the following AppVeyor builds to get an early glimpse:

  • 887: Failed upload due to quota restrictions
  • 888: Successful upload 🎉

Once this is merged, I'll schedule weekly builds

Tons of thanks to @FeodorFitsner for the support in taming curl in a PowerShell context ❤️

/cc @whoisj @KindDragon

@nulltoken
Replace failing CoverityPublisher with plain 7zip and Curl
2f0ed89 
Current build behavior is now:

 - Coverity analysis only happens on scheduled builds
 - Tests aren't run on scheduled builds
 - NuGet package generation only happens on merge commits
 - Coverity/NuGget publishing only happens on the last job of the matrix
@whoisj
Copy link
whoisj commented Jun 8, 2015

Seems fine by me, I cannot think of any hidden downsides. :shipit:

nulltoken added a commit that referenced this pull request Jun 8, 2015
@nulltoken
Merge pull request #1078 from libgit2/ntk/coverity_the_sequel
2a28329 
Replace failing CoverityPublisher with plain 7zip and Curl
@nulltoken nulltoken merged commit 2a28329 into vNext Jun 8, 2015
@nulltoken nulltoken deleted the ntk/coverity_the_sequel branch June 8, 2015 18:27
@nulltoken
Copy link
Member Author

Crontab expression set to 0 0 * * 4

Next runs should occur at

  • 2015-06-11 00:00:00 UTC
  • 2015-06-18 00:00:00 UTC
  • 2015-06-25 00:00:00 UTC
  • 2015-07-02 00:00:00 UTC
  • 2015-07-09 00:00:00 UTC
  • ...

@nulltoken nulltoken mentioned this pull request Jun 9, 2015
Closed
@nulltoken nul 63FB ltoken added this to the v0.22 milestone Jun 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.22
Development

Successfully merging this pull request may close these issues.
2 participants
@nulltoken @whoisj
0