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

Extend github actions for docs PR #7317

Closed

Conversation

shaon72
Copy link

@shaon72 shaon72 commented Aug 8, 2024

Summary

Reduce product docs PR review toil

Ticket Link

Fixes #7280

@mattermost-build
Copy link
Contributor

Hello @shaon72,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@cwarnermm cwarnermm self-requested a review August 12, 2024 14:49
@cwarnermm cwarnermm added the 2: Editor Review Requires review by an editor label Aug 12, 2024
@hanzei hanzei requested a review from toninis August 13, 2024 07:53
@mattermost-build
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@toninis
Copy link
Contributor

toninis commented Sep 9, 2024

I will need some time to test that .
Despite that I do not think I get the scope of this PR . We parse the rst files that changed and then you comment the file paths ?
What's the reason for that . I guess you want to post the whole URL links in order to be efficient .

Is that correct ?

@mattermost-build
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@cwarnermm
Copy link
Member

@shaon72 - This PR produces the list with the files changes (which are also available via GitHub), but doesn't populate the links in the PR summary. Are you interested in updating this PR to include link population?

@cwarnermm cwarnermm added Awaiting Submitter Action Blocked on the author and removed 2: Editor Review Requires review by an editor Lifecycle/1:stale labels Oct 2, 2024
@cwarnermm cwarnermm closed this Oct 17, 2024
@mattermost-build mattermost-build removed the Awaiting Submitter Action Blocked on the author label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Product Docs Request: Extend GitHub Actions
5 participants