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

continue on error for the greeter #3407

Closed
wants to merge 1 commit into from

Conversation

akaszynski
Copy link
Member

I'm implementing the suggestion from this discussion point to simply continue on error until this has been fixed upstream.

@github-actions github-actions bot added the maintenance Low-impact maintenance activity label Oct 4, 2022
Copy link
Member

@tkoyama010 tkoyama010 left a comment

Choose a reason for hiding this comment

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

Since this CI is originally designed to merge Pull Requests even if an error occurs (i.e., it is not related to testing), there is no problem to keep this flag set to True in the future.

@codecov
Copy link

codecov bot commented Oct 4, 2022

Codecov Report

Merging #3407 (c522ff6) into main (e911f28) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3407   +/-   ##
=======================================
  Coverage   94.93%   94.93%           
=======================================
  Files          84       84           
  Lines       18509    18509           
=======================================
  Hits        17572    17572           
  Misses        937      937           

@akaszynski
Copy link
Member Author

Continue on error still means these fail. I'm going to simply comment this out until this is fixed.

@akaszynski
Copy link
Member Author

Simply disabled via disabling-a-workflow.

@akaszynski akaszynski closed this Oct 4, 2022
@akaszynski akaszynski deleted the ci/continue-on-greeter-error branch October 8, 2022 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Low-impact maintenance activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants