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

Incorporate running --check into running --start #349

Merged
merged 1 commit into from
Jan 12, 2024

Conversation

timmullin
Copy link
Contributor

Fixes #348. When running start, first perform a full check for blockers, display them all to the user,
and do not start the migration if any are found.

By submitting pull requests to this repo, I agree to the Contributor License Agreement which can be found at: https://github.com/cpanel/elevate/blob/main/docs/cPanel-CLA.pdf

@cPholloway
Copy link
Contributor

@timmullin - Can you add a changelog entry for this please?

I am going to go ahead and mark this approved pending the changelog entry.

Fixes cpanel#348. When running start, first perform a full
check for blockers, display them all to the user,
and do not start the migration if any are found.
@toddr toddr merged commit dec2401 into cpanel:main Jan 12, 2024
1 check passed
@toddr
Copy link
Member

toddr commented Jan 12, 2024

oof missed the comment. Merged earlier :(

@cPholloway
Copy link
Contributor

It was good to go. The changelog was in place before the merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

/scripts/elevate-cpanel --start should run /scripts/elevate-cpanel --check first
3 participants