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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[program-gen] Emit missing trivia for resources and local variables #15419

Merged
merged 1 commit into from Feb 11, 2024

Conversation

Zaid-Ajaj
Copy link
Contributor

Description

For C#, Python and TypeScript: emit missing comments for local variable declaration
For Go: emit missing comments for local variables and resources

Checklist

  • I have run make tidy to update any new dependencies
  • I have run make lint to verify my code passes the lint check
    • I have formatted my code using gofumpt
  • I have added tests that prove my fix is effective or that my feature works
  • I have run make changelog and committed the changelog/pending/<file> documenting my change
  • Yes, there are changes in this PR that warrants bumping the Pulumi Cloud API version

@Zaid-Ajaj Zaid-Ajaj added the area/codegen SDK-gen, program-gen, convert label Feb 11, 2024
@Zaid-Ajaj Zaid-Ajaj requested a review from a team February 11, 2024 02:05
@pulumi-bot
Copy link
Contributor

Changelog

[uncommitted] (2024-02-11)

Features

  • [programgen/{dotnet,go,nodejs,python}] Emit missing trivia for resources and local variables
    #15419

Copy link
Member

@Frassle Frassle left a comment

Choose a reason for hiding this comment

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

nice

@Zaid-Ajaj Zaid-Ajaj added this pull request to the merge queue Feb 11, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 11, 2024
@Zaid-Ajaj Zaid-Ajaj added this pull request to the merge queue Feb 11, 2024
Merged via the queue into master with commit ed79536 Feb 11, 2024
56 of 57 checks passed
@Zaid-Ajaj Zaid-Ajaj deleted the zaid/emit-more-trivia branch February 11, 2024 17:52
github-merge-queue bot pushed a commit that referenced this pull request Feb 15, 2024
### Features

- [components] Component providers can return output values in their
response to the engine.

- [programgen/{dotnet,go,nodejs,python}] Emit missing trivia for
resources and local variables
  [#15419](#15419)


### Bug Fixes

- [engine] Engine no longer assumes program dependencies are specified
as semantic versions.
  [#15409](#15409)

- [components/go] Component providers now correctly return output values
to the engine.
  [#15405](#15405)


### Miscellaneous

- [protobuf] Split CallRequest into ResourceCallRequest for the
ResourceMonitor service.
  [#15404](#15404)

- [backend/diy] Add a warning that non-project mode will be deprecated
at some point.
  [#15411](#15411)
@justinvp justinvp mentioned this pull request Feb 15, 2024
github-merge-queue bot pushed a commit that referenced this pull request Feb 16, 2024
### Features

- [components] Component providers can return output values in their
response to the engine.
  [#15408](#15408)

- [programgen/{dotnet,go,nodejs,python}] Emit missing trivia for
resources and local variables
  [#15419](#15419)

- [sdk/{nodejs,python}] Publish dev versions of the nodejs and python
SDKs


### Bug Fixes

- [cli] Fix `pulumi destroy` to fill in stack config with the secret
config from state, not fresh secret config.
  [#15432](#15432)

- [engine] Engine no longer assumes program dependencies are specified
as semantic versions.
  [#15409](#15409)

- [components/go] Component providers now correctly return output values
to the engine.
  [#15405](#15405)


### Miscellaneous

- [protobuf] Split CallRequest into ResourceCallRequest for the
ResourceMonitor service.
  [#15404](#15404)

- [backend/diy] Add a warning that non-project mode will be deprecated
at some point.
  [#15411](#15411)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/codegen SDK-gen, program-gen, convert
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants