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

Ignore logging only, not the continue #4243

Merged
merged 2 commits into from
Feb 16, 2024
Merged

Conversation

amourao
Copy link
Contributor

@amourao amourao commented Feb 16, 2024

What's being changed:

Don't skip the continue, only the logging

Review checklist

  • Documentation has been updated, if necessary. Link to changed documentation:
  • Chaos pipeline run or not necessary. Link to pipeline:
  • All new code is covered by tests where it is reasonable.
  • Performance tests have been run or not necessary.

@amourao amourao changed the title Ignore logging only, no the continue Ignore logging only, not the continue Feb 16, 2024
@amourao amourao merged commit a3ad33d into master Feb 16, 2024
13 of 14 checks passed
@amourao amourao deleted the bug/dont-skip-continue branch February 16, 2024 17:49
Copy link

sonarcloud bot commented Feb 16, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

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.

None yet

2 participants