Skip to content

doc-2075: Updating the notify-slack github action script to ping people during their working hours.#7592

Merged
Linus-SpectroCloud merged 20 commits intomasterfrom
doc-2075-gha-call-users
Aug 6, 2025
Merged

doc-2075: Updating the notify-slack github action script to ping people during their working hours.#7592
Linus-SpectroCloud merged 20 commits intomasterfrom
doc-2075-gha-call-users

Conversation

@Linus-SpectroCloud
Copy link
Contributor

@Linus-SpectroCloud Linus-SpectroCloud commented Jul 31, 2025

Describe the Change

This would update the Slack Notification to ping users when they are online for notifications.

github-action-ready-for-review-flowchart

Jira Tickets

🎫 Jira Ticket

Can this PR be backported?

  • Yes. Remember to add the relevant backport labels to your PR.
  • No. Please leave a short comment below about why this PR cannot be backported.

N/A

@Linus-SpectroCloud Linus-SpectroCloud self-assigned this Jul 31, 2025
@Linus-SpectroCloud Linus-SpectroCloud requested a review from a team as a code owner July 31, 2025 14:18
@Linus-SpectroCloud Linus-SpectroCloud added github_actions Pull requests that update GitHub Actions code notify-slack Send notification to slack channel of PR ready for review labels Jul 31, 2025
@netlify
Copy link

netlify bot commented Jul 31, 2025

Deploy Preview for docs-spectrocloud ready!

Name Link
🔨 Latest commit 2d25ada
🔍 Latest deploy log https://app.netlify.com/projects/docs-spectrocloud/deploys/68922ef8797244000879b27a
😎 Deploy Preview https://deploy-preview-7592--docs-spectrocloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Linus-SpectroCloud Linus-SpectroCloud added notify-slack Send notification to slack channel of PR ready for review and removed notify-slack Send notification to slack channel of PR ready for review labels Aug 1, 2025
@spectrocloud spectrocloud deleted a comment from github-actions bot Aug 1, 2025
Copy link
Contributor

@addetz addetz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You've done some excellent work, i made one suggestion which would clean this up a lot and make it easier to modify for new team members too.

caroldelwing
caroldelwing previously approved these changes Aug 1, 2025
Copy link
Contributor

@caroldelwing caroldelwing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super cool! only left one question 🚀

@caroldelwing caroldelwing self-requested a review August 1, 2025 16:12
@Linus-SpectroCloud Linus-SpectroCloud added notify-slack Send notification to slack channel of PR ready for review and removed notify-slack Send notification to slack channel of PR ready for review labels Aug 1, 2025
@Linus-SpectroCloud Linus-SpectroCloud removed the notify-slack Send notification to slack channel of PR ready for review label Aug 1, 2025
addetz
addetz previously approved these changes Aug 4, 2025
Copy link
Contributor

@addetz addetz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FAAB work 👏

* doc-2075: testing changes

* doc-2075: fixed grep conditional

* doc-2075: added Mon-Fri, added option to post @here when no one is on and update to comment
Copy link
Contributor

@addetz addetz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work 👏

@Linus-SpectroCloud Linus-SpectroCloud merged commit 7b793c9 into master Aug 6, 2025
17 checks passed
@Linus-SpectroCloud Linus-SpectroCloud deleted the doc-2075-gha-call-users branch August 6, 2025 13:25
@Linus-SpectroCloud Linus-SpectroCloud added auto-backport Enable backport backport-version-4-2 Backport change to version 4.2 backport-version-4-3 Backport change to version 4.3 backport-version-4-4 Backport change to version 4.4 backport-version-4-5 Backport change to version 4.5 backport-version-4-6 Backport change to version 4.6 backport-version-4-7 Backport change to version 4.7 labels Aug 6, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 6, 2025

✅ Slack reviewers notified at 2025-08-06T13:30:43Z\n<@U065UUXQ8BB> <@U0839DQ1ZA6> <@U07RNMQNWJZ> <@U062EHLFLEQ> <@U08L4LW7X19> \n

@vault-token-factory-spectrocloud
Copy link
Contributor

💔 All backports failed

Status Branch Result
version-4-2 Backport failed because of merge conflicts

You might need to backport the following PRs to version-4-2:
- docs: DOC-1899: ECR S3 Endpoint for Restricted Environments (#7694)
- doc-2073: moved API Audit Logging page and added resources to Audit L… (#7600)
- docs: add gcp permissions (#7554)
- chore: DOC-2016: Upgrade Docusarus to 3.8.1 (#7502)
version-4-3 Backport failed because of merge conflicts

You might need to backport the following PRs to version-4-3:
- docs: DOC-1899: ECR S3 Endpoint for Restricted Environments (#7694)
- doc-2073: moved API Audit Logging page and added resources to Audit L… (#7600)
- docs: add gcp permissions (#7554)
- chore: DOC-2016: Upgrade Docusarus to 3.8.1 (#7502)
version-4-4 Backport failed because of merge conflicts

You might need to backport the following PRs to version-4-4:
- docs: DOC-1899: ECR S3 Endpoint for Restricted Environments (#7694)
- doc-2073: moved API Audit Logging page and added resources to Audit L… (#7600)
- docs: add gcp permissions (#7554)
- chore: DOC-2016: Upgrade Docusarus to 3.8.1 (#7502)
version-4-5 Backport failed because of merge conflicts

You might need to backport the following PRs to version-4-5:
- chore: DOC-2016: Upgrade Docusarus to 3.8.1 (#7502)
version-4-6 Backport failed because of merge conflicts

You might need to backport the following PRs to version-4-6:
- chore: DOC-2016: Upgrade Docusarus to 3.8.1 (#7502)
- docs: PE-7173: Known Issue: Masked Profile Variables Exposed in Edge API Responses (#7473)
version-4-7 Backport failed because of merge conflicts

You might need to backport the following PRs to version-4-7:
- chore: DOC-2016: Upgrade Docusarus to 3.8.1 (#7502)

Manual backport

To create the backport manually run:

backport --pr 7592

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@Linus-SpectroCloud Linus-SpectroCloud removed auto-backport Enable backport backport-version-4-2 Backport change to version 4.2 backport-version-4-3 Backport change to version 4.3 backport-version-4-4 Backport change to version 4.4 backport-version-4-5 Backport change to version 4.5 backport-version-4-6 Backport change to version 4.6 backport-version-4-7 Backport change to version 4.7 notify-slack Send notification to slack channel of PR ready for review labels Aug 6, 2025
@vault-token-factory-spectrocloud
Copy link
Contributor

🎉 This PR is included in version 4.7.10 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants