-
Notifications
You must be signed in to change notification settings - Fork 5
docs(changelog): version 1.5.1 [citest skip] #108
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
Update changelog and .README.html for version 1.5.0 Signed-off-by: Rich Megginson <[email protected]>
Reviewer's GuideThis PR updates the changelog to include the new release notes and refines the HTML README’s code block styling, along with a small documentation typo fix. 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.
Hey @richm - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟡 General issues: 2 issues found
- 🟢 Security: all looks good
- 🟢 Review instructions: all looks good
- 🟢 Testing: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| - ci: ansible-plugin-scan is disabled for now (#87) | ||
| - ci: bump ansible-lint to v25; provide collection requirements for ansible-lint (#90) | ||
| - ci: Check spelling with codespell (#91) | ||
| - ci: Add test plan that runs CI tests and customize it for each role (#92) |
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.
issue (typo): Grammatical error: 'customize' should be 'customizes'.
Use “customizes” to match “runs” in “...that runs CI tests and customizes it...”.
| - ci: Add test plan that runs CI tests and customize it for each role (#92) | |
| - ci: Add test plan that runs CI tests and customizes it for each role (#92) |
| - ci: bump ansible-lint to v25; provide collection requirements for ansible-lint (#90) | ||
| - ci: Check spelling with codespell (#91) | ||
| - ci: Add test plan that runs CI tests and customize it for each role (#92) | ||
| - ci: In test plans, prefix all relate variables with SR_ (#93) |
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.
issue (typo): Typo: 'relate' should be 'related'.
Replace it with 'related'.
| - ci: In test plans, prefix all relate variables with SR_ (#93) | |
| - ci: In test plans, prefix all related variables with SR_ (#93) |
Update changelog and .README.html for version 1.5.1
Signed-off-by: Rich Megginson [email protected]
Summary by Sourcery
Update changelog for version 1.5.1 with the new snapshot manager feature and CI enhancements, and refine README HTML styling and content
New Features:
Enhancements:
Documentation: