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

fix: Include a newline in pulumi whoami's output #14025

Merged
merged 1 commit into from Sep 23, 2023
Merged

Conversation

justinvp
Copy link
Member

Looks like we used to output the newline, but unintentionally changed this in #12400 when refactoring to make it testable. This change fixes the command to include the newline character and updates the associated test.

Fixes #14008

Looks like we used to output the newline, but unintentionally changed this in #12400 when refactoring to make it testable. This change fixes the command to include the newline character and updates the associated test.
@justinvp justinvp requested a review from a team September 23, 2023 06:31
@pulumi-bot
Copy link
Contributor

Changelog

[uncommitted] (2023-09-23)

Bug Fixes

  • [cli] Include a newline in pulumi whoami's output
    #14025

@justinvp justinvp added this pull request to the merge queue Sep 23, 2023
Merged via the queue into master with commit 543e22f Sep 23, 2023
44 checks passed
@justinvp justinvp deleted the justin/whoami branch September 23, 2023 09:48
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.

whoami is missing a newline
3 participants