Skip to content

fix: correct plan/apply divergent error message #5567

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Fab1n
Copy link

@Fab1n Fab1n commented May 5, 2025

what

Change the wording of the error message that gets displayed as comment on PRs where the default branch has diverged from the PR's branch. It switches both mentioned branches to be inline with rebase terminology in git.

Let me know if this is correct or not.

why

The error message is wrong I think and could lead to confusion.

tests

  • I have tested my changes by adapting one necessary test

Also this is only printed string change.

Fab1n added 2 commits May 5, 2025 11:43
Signed-off-by: Fabian Renner <[email protected]>
@github-actions github-actions bot added the go Pull requests that update Go code label May 5, 2025
@dosubot dosubot bot added the bug Something isn't working label May 5, 2025
@Fab1n
Copy link
Author

Fab1n commented May 13, 2025

@nitrocode this would be an easy and quick one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant