Skip to content
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.

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

Dont autoverify on BuildServer #1341

Merged
merged 8 commits into from
Nov 28, 2021
Merged

Conversation

SimonCropp
Copy link
Contributor

@SimonCropp SimonCropp commented Nov 23, 2021

we only want to auto accept changes locally. ie non verified changes should still fail on the build server

#skip-changelog

should prevent breaking changes like #1296

we only want to auto accept changes locally. eg non verified changes should still fail on the build server
@codecov-commenter
Copy link

codecov-commenter commented Nov 25, 2021

Codecov Report

Merging #1341 (91623cd) into main (ca72faa) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1341   +/-   ##
=======================================
  Coverage   82.64%   82.64%           
=======================================
  Files         215      215           
  Lines        7145     7145           
  Branches     1407     1407           
=======================================
  Hits         5905     5905           
  Misses        806      806           
  Partials      434      434           

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 ca72faa...91623cd. Read the comment docs.

@bruno-garcia bruno-garcia merged commit ad9f5ff into main Nov 28, 2021
@bruno-garcia bruno-garcia deleted the AutoVerify(includeBuildServer-false) branch November 28, 2021 17:57
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