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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade pulumi-terraform-bridge to v3.62.0 #152

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jaxxstorm
Copy link
Member

@jaxxstorm jaxxstorm commented Oct 17, 2023

This PR was generated via $ upgrade-provider lbrlabs/pulumi-awscontroltower.


  • Upgrading pulumi-terraform-bridge from v3.61.0 to v3.62.0.

Ellipsis 馃殌 This PR description was created by Ellipsis for commit fabed67.

Summary:

This PR upgrades the pulumi-terraform-bridge to v3.62.0 and updates the upstream-provider-name in /.upgrade-config.yml.

Key points:

  • Upgraded pulumi-terraform-bridge from v3.61.0 to v3.62.0.
  • Updated upstream-provider-name in /.upgrade-config.yml from terraform-provider-awscontroltower to terraform-provider-controltower.

Generated with 鉂わ笍 by ellipsis.dev

@jaxxstorm jaxxstorm self-assigned this Oct 17, 2023
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Looks good to me! Reviewed entire PR up to commit fabed67

Reviewed 8 lines of code across 1 files in 28 second(s).

See details
  • Skipped files: 4 (please contact us to request support for these files): examples/go.mod, examples/go.sum, provider/go.mod, provider/go.sum
  • Confidence threshold: 50%
  • Drafted 1 additional comments.
  • Workflow ID: wflow_y82h3r9MC1ND4NqN
View 1 draft comments

These comments were drafted by Ellipsis, but were filtered out of the final review. They're included here so you can see our internal thought process and help you configure your ellipsis.yaml.

Drafted 1 additional comments

Filtered comment at .upgrade-config.yml:1

Notes: The change in the provider name from 'terraform-provider-awscontroltower' to 'terraform-provider-controltower' might cause issues if the new provider name is not recognized or if it doesn't exist. I need to check if 'terraform-provider-controltower' is a valid provider.

The provider name has been changed from 'terraform-provider-awscontroltower' to 'terraform-provider-controltower'. Please ensure that 'terraform-provider-controltower' is a valid provider.

Confidence changes required: 80%

Score: 0%


Something look wrong? You can customize Ellipsis by editing the ellipsis.yaml for this repository.

Generated with 鉂わ笍 by ellipsis.dev

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.

None yet

1 participant