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

add run method to Pausable #601

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

dev-mlb
Copy link
Collaborator

@dev-mlb dev-mlb commented Oct 2, 2023

No description provided.

@dev-mlb dev-mlb requested a review from jpdahlke October 2, 2023 21:28
@jpdahlke jpdahlke added this to the v8.0.0-M7 milestone Oct 2, 2023
@jpdahlke jpdahlke added the tech-debt Low-impact cleanup and upkeep label Oct 2, 2023
@cfkoehler cfkoehler self-requested a review October 3, 2023 14:18
@jpdahlke jpdahlke merged commit 93f80a7 into NationalSecurityAgency:master Oct 3, 2023
8 of 9 checks passed
@dev-mlb dev-mlb deleted the pausable-run branch October 3, 2023 15:51
@drivenflywheel
Copy link
Collaborator

Sonarqube has way too many bad recommendations. The violated rule isn't applicable to abstract classes, but that doesn't stop it from false positives

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-debt Low-impact cleanup and upkeep
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants