Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Allowexeca return values in$ template expressions#528

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@aaronccasanova
Copy link
Contributor

Closes#523

This PR adds support for interpolatingExecaReturnValue andExecaSyncReturnValue in$ template expressions:

Asynchronous

import{$}from'execa';constrainbows=await$`echo rainbows`;await$({stdio:'inherit'})`echo${rainbows} unicorns`;//=> 'rainbows unicorns'

Synchronous

import{$}from'execa';constrainbows=$.sync`echo rainbows`;$({stdio:'inherit'}).sync`echo${rainbows} unicorns`;//=> 'rainbows unicorns'

ntnyq reacted with hooray emojiehmicky reacted with heart emoji
@aaronccasanova
Copy link
ContributorAuthor

@sindresorhus@ehmicky I'll update the JSDoc types and README documentation after sign off on the implementation.

@ehmicky
Copy link
Collaborator

This is looking great@aaronccasanova, thanks! 🎉

@ehmicky
Copy link
Collaborator

This looks good to me so far! 👍

@aaronccasanova
Copy link
ContributorAuthor

Great, I'll start on the docs and open the PR shortly!

ehmicky reacted with rocket emoji

@aaronccasanovaaaronccasanova marked this pull request as ready for reviewMarch 5, 2023 21:37
@ehmickyehmicky self-requested a reviewMarch 5, 2023 21:50
Copy link
Collaborator

@ehmickyehmicky left a comment

Choose a reason for hiding this comment

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

🚀

@sindresorhussindresorhus merged commit58ea251 intosindresorhus:mainMar 6, 2023
@sindresorhus
Copy link
Owner

Looks great 👍

@aaronccasanovaaaronccasanova deleted the execa-result-expressions branchMarch 6, 2023 05:58
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@ehmickyehmickyehmicky approved these changes

@sindresorhussindresorhusAwaiting requested review from sindresorhus

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

${childProcessResult} shortcut notation

3 participants

@aaronccasanova@ehmicky@sindresorhus
[8]ページ先頭
©2009-2025 Movatter.jp