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(review-apps) wording changes #922

Merged
merged 5 commits into from
Mar 30, 2023
Merged

Conversation

yohann-bacha
Copy link
Contributor

No need to add changelog entries

Fixes #921

I also changed the way the integration settings are displayed, using go templates. I think this is much clearer than doing it programmatically, if you have some returns let me know

(sorry etienne, github does not allow me to remove you from the reviewers)

@yohann-bacha yohann-bacha self-assigned this Mar 29, 2023
Copy link

@guillaume-sc guillaume-sc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the wording with regard to what Product Management provided.

cmd/integration_link.go Outdated Show resolved Hide resolved
cmd/integration_link.go Outdated Show resolved Hide resolved
Copy link

@guillaume-sc guillaume-sc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The text I asked you to use in place of the existing one has no meaning. Please hold on this PR until we craft a correct error message with Product Management.

cmd/integration_link.go Outdated Show resolved Hide resolved
cmd/integration_link.go Outdated Show resolved Hide resolved
Signed-off-by: Yohann Bacha <[email protected]>
@yohann-bacha yohann-bacha merged commit 2607824 into master Mar 30, 2023
@yohann-bacha yohann-bacha deleted the feat/921/wording-changes branch March 30, 2023 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change messages displayed when allowing automatic review app creation from forks
2 participants