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

Adding @actions/exec to github-script #178

Merged
merged 8 commits into from Aug 19, 2021

Conversation

bhavanakonchada
Copy link

The @actions/exec package is useful to execute tools in a cross platform way. This package is now available to the script using the exec variable.

@bhavanakonchada bhavanakonchada requested a review from a team as a code owner August 11, 2021 20:57
Copy link
Member

@joshmgross joshmgross left a comment

Choose a reason for hiding this comment

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

👋 Hey, thanks for this contribution. It looks like a large change was made to package-lock.json switching from lockfileVersion 2 to 1 - https://docs.npmjs.com/cli/v7/configuring-npm/package-lock-json#lockfileversion

Since we're already on a newer version, I'd prefer not go back to version 1. Could you update your version of npm and re-run npm install?

I'd be happy to help with that change if you'd like.

@bhavanakonchada
Copy link
Author

👋 Hey, thanks for this contribution. It looks like a large change was made to package-lock.json switching from lockfileVersion 2 to 1 - https://docs.npmjs.com/cli/v7/configuring-npm/package-lock-json#lockfileversion

Since we're already on a newer version, I'd prefer not go back to version 1. Could you update your version of npm and re-run npm install?

I'd be happy to help with that change if you'd like.

@joshmgross thanks so much for taking a keen look at the PR, updated npm as you suggested to ensure package-lock.json is now at version 2.

@bhavanakonchada
Copy link
Author

Hi @joshmgross , do you believe this PR can be merged, after the checks passing and incorporating feedback ?

@joshmgross joshmgross merged commit e02270e into actions:main Aug 19, 2021
@joshmgross
Copy link
Member

👋 Hey @bhavanakonchada, thanks for your contribution!

This is now available in https://github.com/actions/github-script/releases/tag/v4.1.0, which can be accessed with the following refs in your workflow:

  • main
  • v4.1.0
  • v4.1
  • v4

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

2 participants