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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change await log type to cloud-ready-check lib #1855
Conversation
Does the PR have any schema changes?Looking good! No breaking changes found. |
2 similar comments
Does the PR have any schema changes?Looking good! No breaking changes found. |
Does the PR have any schema changes?Looking good! No breaking changes found. |
The cloud-ready-check library includes an identical logging.Message type definition, so remove the type from the provider and update all references to use the external type instead.
95c9674
to
4166b9f
Compare
Does the PR have any schema changes?Looking good! No breaking changes found. |
@@ -1,4 +1,4 @@ | |||
// Copyright 2016-2019, Pulumi Corporation. | |||
// Copyright 2016-2022, Pulumi Corporation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this file be in cloud-ready-checks instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this logic is more specific to provider logging, so I decided to leave it here. The await library returns the messages and it's up to the client how to handle them.
@@ -1,4 +1,4 @@ | |||
// Copyright 2016-2019, Pulumi Corporation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here
@@ -1,4 +1,4 @@ | |||
// Copyright 2016-2019, Pulumi Corporation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here.
Proposed changes
The cloud-ready-check library includes an identical
logging.Message type definition, so remove the type
from the provider and update all references to use the
external type instead.
Related issues (optional)