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

feat: add workflow-name-delimiter flag #3815

Closed
wants to merge 1 commit into from

Conversation

yorik
Copy link

@yorik yorik commented Oct 5, 2023

what

Add workflow-name-delimiter flag to allow change delimiter in workflow name from default :.

why

To use something compatible with terraform-provider-github and avoid integrations/terraform-provider-github#1531.

tests

references

closes #3814

@yorik yorik requested a review from a team as a code owner October 5, 2023 16:24
@github-actions github-actions bot added the go Pull requests that update Go code label Oct 5, 2023
@jamengual
Copy link
Contributor

I'm no so keen to add a workaround on the atlantis code because something else have a bug, I think we should wait until they fix it

@jamengual jamengual added needs discussion Large change that needs review from community/maintainers do-not-merge labels Oct 5, 2023
@yorik
Copy link
Author

yorik commented Oct 5, 2023

I do understand that, but they fixing the problem since February: it will be breaking change for them unfortunately, so I thought that improve atlantis would easier for everyone.

@jamengual
Copy link
Contributor

I do understand that, but they have been fixing the problem since February: it will be breaking change for them unfortunately, so I thought that improve atlantis would easier for everyone.

I get that, but who will then create the PR to revert this once this is fixed?

This is a Terraform issue on a provider which is 3rd levels down from Atlantis code, we can't use Atlantis as a workaround for code that Atlantis is not responsible for.

This is good documentation for other people with similar issues so they know why this happens.

@jamengual jamengual closed this Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge go Pull requests that update Go code needs discussion Large change that needs review from community/maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Atlantis action name format is incompatible with terraform-provider-github
2 participants