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

remove deprecated runcommand and main-runner #568

Merged
merged 5 commits into from
Dec 12, 2023

Conversation

dev-mlb
Copy link
Collaborator

@dev-mlb dev-mlb commented Sep 9, 2023

Will have merge conflicts with #561

@dev-mlb dev-mlb added tech-debt Low-impact cleanup and upkeep integration Effort is required from downstream customers labels Sep 9, 2023
@dev-mlb dev-mlb added this to the v8.0.0-M6 milestone Sep 9, 2023
drivenflywheel
drivenflywheel previously approved these changes Sep 12, 2023
Copy link
Collaborator

@jpdahlke jpdahlke left a comment

Choose a reason for hiding this comment

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

The commons-cli dependency can get removed with this as well it seems?

@jpdahlke jpdahlke modified the milestones: v8.0.0-M8, v8.0.0-M9 Oct 5, 2023
@cfkoehler cfkoehler modified the milestones: v8.0.0-M9, v8.0.0-M10 Nov 7, 2023
@jpdahlke jpdahlke modified the milestones: v8.0.0-M10, v8.0.0-M11 Nov 14, 2023
@jpdahlke jpdahlke self-requested a review December 12, 2023 14:52
@cfkoehler cfkoehler self-requested a review December 12, 2023 15:08
@jpdahlke jpdahlke merged commit 244e740 into NationalSecurityAgency:master Dec 12, 2023
9 checks passed
@dev-mlb dev-mlb deleted the deprecated-run-main branch December 13, 2023 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration Effort is required from downstream customers tech-debt Low-impact cleanup and upkeep
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants