-
Notifications
You must be signed in to change notification settings - Fork 71
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
feat: Add extra options #117
Changes from 3 commits
3c9133e
a6afc94
d04da30
ab8dfec
28142aa
67c526b
0c82a26
3f2b981
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,45 @@ | ||
name: 'Application with Extra Options' | ||
on: | ||
pull_request: | ||
push: | ||
branches: | ||
- main | ||
- master | ||
|
||
jobs: | ||
test-workflow: | ||
runs-on: ${{ matrix.os }} | ||
name: Testing Pulumi ${{ matrix.command }} on ${{ matrix.os }} | ||
strategy: | ||
matrix: | ||
command: [ up, refresh, destroy, preview ] | ||
os: [ubuntu-latest, macos-latest, windows-latest] | ||
fail-fast: false | ||
steps: | ||
- uses: actions/checkout@v2 | ||
- uses: actions/setup-node@v1 | ||
with: | ||
node-version: 14.x | ||
- name: Install pulumi | ||
uses: pulumi/action-install-pulumi-cli@v1.0.1 | ||
- name: Create local stack | ||
run: | | ||
pulumi login --local | ||
pulumi stack init dev | ||
working-directory: .github/test-stacks/nodejs | ||
env: | ||
PULUMI_CONFIG_PASSPHRASE: not-a-secret | ||
- run: npm install | ||
working-directory: .github/test-stacks/nodejs | ||
- uses: ./ | ||
env: | ||
PULUMI_CONFIG_PASSPHRASE: not-a-secret | ||
with: | ||
command: ${{ matrix.command }} | ||
cloud-url: file://~ | ||
stack-name: dev | ||
work-dir: .github/test-stacks/nodejs | ||
parallel: 3 | ||
message: this-is-just-a-test | ||
expect-no-changes: false | ||
diff: true | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Missing |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -53,12 +53,35 @@ The action can be configured with the following arguments: | |
equivalent of what would be passed to the `pulumi login` command. The action | ||
will login to the appropriate backend on your behalf provided it is configured | ||
with the correct access credentials for that backend. | ||
|
||
- `comment-on-pr` - (optional) If `true`, then the action will add the results | ||
of the Pulumi action to the PR | ||
|
||
- `github-token` - (required if comment-on-pr) A GitHub token that has access | ||
levels to allow the Action to comment on a PR. | ||
|
||
### Extra options | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should add a small comment here to suggest that these extra options, are the equivalent of the command line args |
||
|
||
- `parallel` - (optional) Allow P resource operations to run in parallel at once | ||
(1 for no parallelism). Defaults to unbounded. | ||
|
||
- `message` - (optional) Optional message to associate with the update operation | ||
|
||
- `expect-no-changes` - (optional) Return an error if any changes occur during | ||
this update | ||
|
||
- `diff` - (optional) Display operation as a rich diff showing the overall | ||
change | ||
|
||
- `replace` - (optional) Specify resources to replace. Multiple resources can be | ||
specified one per line | ||
|
||
- `target` - (optional) Specify a single resource URN to update. Other resources | ||
will not be updated. Multiple resources can be specified one per line | ||
|
||
- `target-dependents` - (optional) Allows updating of dependent targets | ||
discovered but not specified in target. | ||
|
||
By default, this action will try to authenticate Pulumi with the | ||
[Pulumi SaaS](https://app.pulumi.com/). If you have not specified a | ||
`PULUMI_ACCESS_TOKEN` then you will need to specify an alternative backend via | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -27,6 +27,30 @@ inputs: | |
description: 'A cloud URL to log in to' | ||
required: false | ||
default: '' | ||
parallel: | ||
description: 'Allow P resource operations to run in parallel at once (1 for no parallelism). Defaults to unbounded.' | ||
required: false | ||
default: '2147483647' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this relates to what the automation api does - this isn't actually the number we pass, this is to document what we can do |
||
message: | ||
description: 'Optional message to associate with the update operation' | ||
required: false | ||
default: '' | ||
expect-no-changes: | ||
description: 'Return an error if any changes occur during this update' | ||
required: false | ||
diff: | ||
description: 'Display operation as a rich diff showing the overall change' | ||
required: false | ||
replace: | ||
description: 'Specify resources to replace. Multiple resources can be specified one per line' | ||
required: false | ||
target: | ||
description: 'Specify a single resource URN to update. Other resources will not be updated. Multiple resources can be specified one per line' | ||
required: false | ||
target-dependents: | ||
description: 'Allows updating of dependent targets discovered but not specified in target.' | ||
required: false | ||
default: 'false' | ||
outputs: | ||
output: | ||
description: Output from running command | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prettier/@typescript-eslint
has been merged intoprettier
in eslint-config-prettier 8.0.0. See: https://github.com/prettier/eslint-config-prettier/blob/main/CHANGELOG.md#version-800-2021-02-21