Skip to content

Conversation

@aacotroneo
Copy link
Contributor

Proposed changes

Unreleased product version (optional)

Related issues (optional)

@claude
Copy link
Contributor

claude bot commented Nov 21, 2025

Documentation Review

Thank you for updating the AWS Organizations Tag Policies resource type mappings. I've reviewed the changes and found one potential issue that needs verification:

Issues Found

Line 95: Incorrect provider namespace for event-integration

| app-integrations:event-integration | aws:appconfig/eventIntegration:EventIntegration |

The mapping shows aws:appconfig/eventIntegration:EventIntegration, but based on the pattern of the line above it (app-integrations:data-integrationaws:appintegrations/dataIntegration:DataIntegration), this should likely be:

| app-integrations:event-integration | aws:appintegrations/eventIntegration:EventIntegration |

Please verify this mapping against the AWS Pulumi provider documentation to ensure it's correct. EventIntegration should likely be under the appintegrations provider namespace, not appconfig.

Additional Notes

The other new mappings added (aoss:collection, ec2:client-vpn-endpoint, ec2:instance-connect-endpoint, ec2:verified-access-* resources, ecr-public:repository, lightsail:LoadBalancer, and notifications-contacts:emailcontact) appear to follow the correct format and are alphabetically sorted within their sections.


If you'd like me to help verify the correct Pulumi provider resource types or make additional changes, mention me (@claude) in a comment.

@pulumi-bot
Copy link
Collaborator

@aacotroneo aacotroneo merged commit e258cd8 into master Nov 21, 2025
12 checks passed
@aacotroneo aacotroneo deleted the aa/update-types branch November 21, 2025 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants