-
Notifications
You must be signed in to change notification settings - Fork 8
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
Manual Only Mode #24
Comments
are you using the pipeline for anything else? Just the production sync? Could probably just have a separate pipelines file thats significantly reduced and controlled by a yeoman param |
mostly production sync, and hot fix package generation (which is basically none of the PRs if we have a big project, and very few relative to standard release deploys) kind of thing this is more of an environmental variable thing, what would the benefit of having a reduced pipeline file and related scripts be? |
Mainly just to take the option off the table if it shouldn't be done due to governance/change management. But if it's still used in some instances, then a ENV makes more sense. DISABLE_PR_PIPELINE? There's also already an exit check on the PR pipeline that's hardcoded to make sure it only runs on Downside is that requires an additional one time setup/upgrade step |
In a few cases working on projects where all the deployments are handled by a separate process. Would be nice to have a config variable to disable the pull request automation so we don't generate extra build time.
if [ ! -z "$MANUAL_ONLY" ]; then ./sync.sh
The text was updated successfully, but these errors were encountered: