-
Notifications
You must be signed in to change notification settings - Fork 142
fix: some typos in the success migrator comment. #4096
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
Conversation
|
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
Reviewer's GuideThis PR refines the success notification in the migrator workflow by correcting typos, clarifying messaging, updating branch references, and standardizing user placeholders for consistency. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request fixes typos and improves the wording in the success migrator comment that appears when a migration is completed. The changes focus on making the automated message more clear and professional.
Key changes:
- Corrected terminology from "PR" to "pull request" for clarity
- Fixed user reference to use the original PR author instead of the triggering user
- Improved the structure and flow of the migration completion message
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
@pyansys-ci-bot LGTM. |
pyansys-ci-bot
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4096 +/- ##
==========================================
+ Coverage 91.82% 91.85% +0.03%
==========================================
Files 187 187
Lines 15033 15033
==========================================
+ Hits 13804 13809 +5
+ Misses 1229 1224 -5 🚀 New features to boost your workflow:
|

Description
As the title.
Issue linked
NA
Checklist
draftif it is not ready to be reviewed yet.feat: adding new MAPDL command)Summary by Sourcery
Correct typos and clarify messaging in the GitHub Actions migrator workflow’s success comment.
CI: