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: allow upper case in doc-deploy-changelog #497

Merged
merged 5 commits into from
Jun 5, 2024

Conversation

SMoraisAnsys
Copy link
Contributor

Closes #496

@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@github-actions github-actions bot added the enhancement General improvements to existing features label Jun 4, 2024
@RobPasMue RobPasMue changed the title feature: allow upper case in doc-deploy-changelog feat: allow upper case in doc-deploy-changelog Jun 4, 2024
@SMoraisAnsys SMoraisAnsys marked this pull request as ready for review June 4, 2024 15:02
@SMoraisAnsys SMoraisAnsys requested a review from a team as a code owner June 4, 2024 15:02
@SMoraisAnsys
Copy link
Contributor Author

Tested it while enabling the use of upper-case and it seems to work well

image

Should I also add a type to the commit added when creating the fragment ? We would have by default, "chore: adding changelog entry..." instead of "Adding changelog entry..."

Copy link
Contributor

@MaxJPRey MaxJPRey left a comment

Choose a reason for hiding this comment

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

Thanks @SMoraisAnsys . I will be able to proceed with my upper case habits regarding PRs' titles.

Co-authored-by: Revathy Venugopal <[email protected]>
@RobPasMue
Copy link
Member

Can we merge this?

@SMoraisAnsys
Copy link
Contributor Author

Can we merge this?

Yes !

@RobPasMue RobPasMue merged commit f20b62a into main Jun 5, 2024
16 checks passed
@RobPasMue RobPasMue deleted the feat/deploy-changelog-upper-cases branch June 5, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement General improvements to existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Doc-deploy-changelog action does not work well with commit-style action when use-upper-case is set to True
5 participants