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

Flip --incompatible_merge_fixed_and_default_shell_env #19318

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Aug 24, 2023

Fixes #5980
Closes #19317

RELNOTES[INC]: --incompatible_merge_fixed_and_default_shell_env is flipped to true. See #19317 for details.

@fmeum fmeum force-pushed the flip-incompatible_merge_fixed_and_default_shell_env branch 2 times, most recently from ddb0c68 to 8c0df7f Compare August 24, 2023 08:19
@fmeum fmeum changed the title Flip --incompatible_merge_fixed_and_default_shell_env Flip --incompatible_merge_fixed_and_default_shell_env Aug 24, 2023
@fmeum fmeum force-pushed the flip-incompatible_merge_fixed_and_default_shell_env branch from 8c0df7f to 79c7f22 Compare August 25, 2023 10:26
@fmeum
Copy link
Collaborator Author

fmeum commented Aug 25, 2023

@fmeum fmeum force-pushed the flip-incompatible_merge_fixed_and_default_shell_env branch from 79c7f22 to 545a5ab Compare August 25, 2023 10:27
@fmeum fmeum marked this pull request as ready for review August 25, 2023 10:27
@fmeum fmeum requested a review from comius August 25, 2023 10:27
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Aug 25, 2023
@fmeum
Copy link
Collaborator Author

fmeum commented Aug 25, 2023

@comius In addition to flipping the flag, I also updated the docs.

@comius comius added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Aug 25, 2023
@comius
Copy link
Contributor

comius commented Aug 25, 2023

Thanks! I think this is low-risk for internal breakages. After import I'll let nightly testing figure that out. If it does, there might be a rollback in following days.

@iancha1992 iancha1992 added the team-Rules-API API for writing rules/aspects: providers, runfiles, actions, artifacts label Aug 25, 2023
@github-actions github-actions bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Aug 28, 2023
@fmeum fmeum deleted the flip-incompatible_merge_fixed_and_default_shell_env branch August 28, 2023 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Rules-API API for writing rules/aspects: providers, runfiles, actions, artifacts
Projects
None yet
3 participants