-
Notifications
You must be signed in to change notification settings - Fork 4
Add section for SAP-internal usage of pull request automation #18
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -93,6 +93,10 @@ This project is open to feature requests/suggestions, bug reports etc. via [GitH | |
|
|
||
| If you find any bug that may be a security problem, please follow our instructions at [in our security policy](https://github.com/SAP/ai-assisted-github-actions/security/policy) on how to report it. Please do not create GitHub issues for security-related doubts or problems. | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The phrase 'following our instructions' in the security policy link text is slightly redundant. Consider simplifying it to enhance readability: Please follow our [security policy](https://github.com/SAP/ai-assisted-github-actions/security/policy) on how to report security bugs. |
||
|
|
||
| ## SAP-internal Usage | ||
|
|
||
| As a SAP employee, please check out and use our [internal integrated solution for pull request automation](https://url.sap/2qqvl0). | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In line 6, the phrase 'check out and use our [internal integrated solution for pull request automation]' can be streamlined for conciseness. Consider using 'use our [internal pull request automation solution]' which is clearer and avoids redundancy. As a SAP employee, please use our [internal pull request automation solution](https://url.sap/2qqvl0).This phrasing reduces complexity and improves the flow of information. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In the section for SAP-internal usage, 'check out and use our' could be rephrased to 'explore and leverage our' to align the language with professional tone: As a SAP employee, please explore and leverage our [internal integrated solution for pull request automation](https://url.sap/2qqvl0). |
||
|
|
||
| ## Code of Conduct | ||
|
|
||
| We as members, contributors, and leaders pledge to make participation in our community a harassment-free experience for everyone. By participating in this project, you agree to abide by its [Code of Conduct](https://github.com/SAP/.github/blob/main/CODE_OF_CONDUCT.md) at all times. | ||
|
|
||
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.
The phrase 'please follow our instructions at [in our security policy]' in line 2 contains a typo and inconsistent phrasing. It could be improved for clarity by changing to 'please follow our [security policy instructions]' for a more direct reference to the document.
This adjustment enhances readability and directs users more effectively to the policy link.