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

Add jsonParse to nodejs sdk #11735

Merged
merged 1 commit into from Dec 28, 2022
Merged

Add jsonParse to nodejs sdk #11735

merged 1 commit into from Dec 28, 2022

Conversation

Frassle
Copy link
Member

@Frassle Frassle commented Dec 26, 2022

Description

Partner method to jsonStringify.

Checklist

  • 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 Service API version

@pulumi-bot
Copy link
Contributor

pulumi-bot commented Dec 26, 2022

Changelog

[uncommitted] (2022-12-28)

Features

  • [sdk/nodejs] Add output jsonParse using JSON.parse.
    #11735

Copy link
Member

@justinvp justinvp left a comment

Choose a reason for hiding this comment

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

LGTM otherwise

sdk/nodejs/output.ts Outdated Show resolved Hide resolved
@Frassle
Copy link
Member Author

Frassle commented Dec 28, 2022

bors merge

@bors
Copy link
Contributor

bors bot commented Dec 28, 2022

Build succeeded:

@bors bors bot merged commit 497ebc5 into master Dec 28, 2022
@bors bors bot deleted the fraser/nodejsJsonParse branch December 28, 2022 10:23
@justinvp justinvp mentioned this pull request Jan 21, 2023
31 tasks
@lukehoban lukehoban mentioned this pull request Mar 27, 2023
19 tasks
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

3 participants