Skip to content

Incident_io README changes with new shortcode and webhook_config.json#23194

Merged
moreauowen merged 4 commits intomasterfrom
moreauowen/TXP-72_incident-io-docs-uses-new-shortcode
Apr 16, 2026
Merged

Incident_io README changes with new shortcode and webhook_config.json#23194
moreauowen merged 4 commits intomasterfrom
moreauowen/TXP-72_incident-io-docs-uses-new-shortcode

Conversation

@moreauowen
Copy link
Copy Markdown
Contributor

@moreauowen moreauowen commented Apr 6, 2026 •
edited
Loading

What does this PR do?

Attempting to update Incident.IO docs to use new shortcode

Motivation

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@moreauowen moreauowen mentioned this pull request Apr 7, 2026
Merged
1 task
@moreauowen moreauowen changed the title Test Incident_io README changes with new shortcode Test Incident_io README changes with new shortcode and webhook_config.json Apr 7, 2026
@moreauowen moreauowen added the assets/no-deploy Prevents APW from deploying this PR in staging label Apr 7, 2026
@goncalves-guilherme goncalves-guilherme added the assets/no-dry-run Run asset publishing github checks in staging label Apr 8, 2026
@goncalves-guilherme
Copy link
Copy Markdown

goncalves-guilherme commented Apr 8, 2026

Added assets/no-dry-run label just to confirm if my validation step will appear up on the validation list.

@moreauowen moreauowen removed the assets/no-deploy Prevents APW from deploying this PR in staging label Apr 8, 2026
@moreauowen moreauowen marked this pull request as ready for review April 10, 2026 14:38
@moreauowen moreauowen requested review from a team as code owners April 10, 2026 14:38
git-thuerk-done
git-thuerk-done previously approved these changes Apr 10, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@git-thuerk-done git-thuerk-done left a comment

Choose a reason for hiding this comment

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

reviewed changes on local preview and works great!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 13, 2026

⚠️ Recommendation: Add qa/skip-qa label

This PR does not modify any files shipped with the agent.

To help streamline the release process, please consider adding the qa/skip-qa label if these changes do not require QA testing.

@moreauowen moreauowen added qa/skip-qa Automatically skip this PR for the next QA and removed assets/no-dry-run Run asset publishing github checks in staging labels Apr 13, 2026
@moreauowen moreauowen changed the title Test Incident_io README changes with new shortcode and webhook_config.json Incident_io README changes with new shortcode and webhook_config.json Apr 13, 2026
@moreauowen moreauowen force-pushed the moreauowen/TXP-72_incident-io-docs-uses-new-shortcode branch from 5262ab9 to 30999b6 Compare April 15, 2026 12:52
@temporal-github-worker-1 temporal-github-worker-1 Bot dismissed stale reviews from goncalves-guilherme and git-thuerk-done April 15, 2026 12:52

Review from goncalves-guilherme is dismissed. Related teams and files:

  • saas-integrations
    • incident_io/README.md
    • incident_io/assets/webhook_config.json

Review from git-thuerk-done is dismissed. Related teams and files:

  • documentation
    • incident_io/README.md
@dd-octo-sts
Copy link
Copy Markdown
Contributor

dd-octo-sts Bot commented Apr 16, 2026

Validation Report

All 20 validations passed.

Show details
Validation Description Status
agent-reqs Verify check versions match the Agent requirements file ✅
ci Validate CI configuration and Codecov settings ✅
codeowners Validate every integration has a CODEOWNERS entry ✅
config Validate default configuration files against spec.yaml ✅
dep Verify dependency pins are consistent and Agent-compatible ✅
http Validate integrations use the HTTP wrapper correctly ✅
imports Validate check imports do not use deprecated modules ✅
integration-style Validate check code style conventions ✅
jmx-metrics Validate JMX metrics definition files and config ✅
labeler Validate PR labeler config matches integration directories ✅
legacy-signature Validate no integration uses the legacy Agent check signature ✅
license-headers Validate Python files have proper license headers ✅
licenses Validate third-party license attribution list ✅
metadata Validate metadata.csv metric definitions ✅
models Validate configuration data models match spec.yaml ✅
openmetrics Validate OpenMetrics integrations disable the metric limit ✅
package Validate Python package metadata and naming ✅
readmes Validate README files have required sections ✅
saved-views Validate saved view JSON file structure and fields ✅
version Validate version consistency between package and changelog ✅

View full run

@moreauowen moreauowen added this pull request to the merge queue Apr 16, 2026
Merged via the queue into master with commit a98abef Apr 16, 2026
47 of 48 checks passed
@moreauowen moreauowen deleted the moreauowen/TXP-72_incident-io-docs-uses-new-shortcode branch April 16, 2026 17:44
@dd-octo-sts dd-octo-sts Bot added this to the 7.79.0 milestone Apr 16, 2026
github-actions Bot pushed a commit that referenced this pull request Apr 16, 2026
@github-merge-queue
Incident_io README changes with new shortcode and webhook_config.json (…
4053a41 
…#23194)

* Test Incident_io README changes with new shortcode

* Add webhook_config.json a98abef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@goncalves-guilherme goncalves-guilherme goncalves-guilherme approved these changes

@git-thuerk-done git-thuerk-done git-thuerk-done left review comments

Assignees

No one assigned

Labels

docs/review-requested documentation ecosystems/review-requested integration/incident_io product/review-requested qa/skip-qa Automatically skip this PR for the next QA team/documentation team/saas-integrations

Projects

None yet

Milestone

7.79.0

Development

Successfully merging this pull request may close these issues.

3 participants

@moreauowen @goncalves-guilherme @git-thuerk-done