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

ci(reviewBot): implement review bot in python to verify reviews #3235

Merged
merged 13 commits into from
May 29, 2024

Conversation

Slashgear
Copy link
Contributor

@Slashgear Slashgear commented May 28, 2024

New bot in python to check which docs are due for review, and post a list to the Slack channel (every Monday at 8am)

If you want to test (without triggering a message to Slack every time)

You need to modify the file so that it works from local:

  • go to to documentations/packages/docs/content/bin/check-review-dates.py
  • Comment out or delete lines 122 and 123 (to avoid sending the message to Slack)
  • In line 119, change the single dot to a double dot (docs_to_review = process_files(".."))
  • run python check-review-dates.py - the categorized list of docs to review should print to the terminal.

.github/workflows/docs-review-bot.yml Outdated Show resolved Hide resolved
bin/check-review-dates.py Outdated Show resolved Hide resolved
@Slashgear Slashgear marked this pull request as ready for review May 29, 2024 07:49
@RoRoJ RoRoJ added priority: medium Maintenance PRs to fix several visible mistakes status: doc review Waiting for doc team review labels May 29, 2024
@RoRoJ RoRoJ merged commit 854d5b5 into main May 29, 2024
4 checks passed
@RoRoJ RoRoJ deleted the docs-review-bot branch May 29, 2024 12:39
RoRoJ added a commit that referenced this pull request Jun 14, 2024
ci(reviewBot): implement review bot in python to verify reviews

Co-authored-by: Antoine Caron <[email protected]>
Co-authored-by: Rowena <[email protected]>
Co-authored-by: Rowena Jones <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium Maintenance PRs to fix several visible mistakes status: doc review Waiting for doc team review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants