Skip to content

fix: all errors now have error codes#36934

Merged
mergify[bot] merged 11 commits intomainfrom
huijbers/error-codes
Mar 16, 2026
Merged

fix: all errors now have error codes#36934
mergify[bot] merged 11 commits intomainfrom
huijbers/error-codes

Conversation

@rix0rrr
Copy link
Copy Markdown
Contributor

@rix0rrr rix0rrr commented Feb 10, 2026

We would like to build a knowledge base of common errors, so that resolution strategies can be described and shared. To that end, it's easier if all errors get an error code.

Use AI to invent an error code for all errors.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@rix0rrr rix0rrr requested a review from a team as a code owner February 10, 2026 16:45
@github-actions github-actions bot added the p2 label Feb 10, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team February 10, 2026 16:45
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Feb 10, 2026
Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

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

(This review is outdated)

@rix0rrr rix0rrr force-pushed the huijbers/error-codes branch from fafeeb2 to 3d55a08 Compare March 6, 2026 12:37
@rix0rrr rix0rrr added the pr-linter/exempt-integ-test The PR linter will not require integ test changes label Mar 6, 2026
@rix0rrr rix0rrr changed the title feat: all errors now have error codes fix: all errors now have error codes Mar 6, 2026
@rix0rrr rix0rrr added the pr-linter/exempt-test The PR linter will not require test changes label Mar 6, 2026
@aws-cdk-automation aws-cdk-automation dismissed their stale review March 6, 2026 13:09

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

@rix0rrr rix0rrr force-pushed the huijbers/error-codes branch from 3d55a08 to 87df87d Compare March 9, 2026 10:19
@rix0rrr rix0rrr force-pushed the huijbers/error-codes branch from 87df87d to c3bd81d Compare March 9, 2026 14:19
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Mar 16, 2026

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit 408c12f into main Mar 16, 2026
17 of 19 checks passed
@mergify mergify bot deleted the huijbers/error-codes branch March 16, 2026 14:01
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Mar 16, 2026

Merge Queue Status

  • Entered queue2026-03-16 14:01 UTC · Rule: default-squash
  • Checks passed · in-place
  • Merged2026-03-16 14:01 UTC · at 7cf4a59dcc6d9f81d5eff25970407ea3cbc99571

This pull request spent 18 seconds in the queue, including 1 second running CI.

Required conditions to merge
  • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • any of [🛡 GitHub branch protection]:
    • check-success = validate-pr
    • check-neutral = validate-pr
    • check-skipped = validate-pr
  • any of [🛡 GitHub branch protection]:
    • check-success = build
    • check-neutral = build
    • check-skipped = build

@github-actions
Copy link
Copy Markdown
Contributor

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contribution/core This is a PR that came from AWS. p2 pr-linter/exempt-integ-test The PR linter will not require integ test changes pr-linter/exempt-test The PR linter will not require test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants