Skip to content

Add PR number to output when triggered by push #13

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.

Sign up for GitHub

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add PR number to output when triggered by push #13

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,10 @@ jobs:
auto-merge: true # If `true`, will try to automatically merge the pull requests.
```

### Output
When triggered by a push event, the pull request number which is created or found to
already exist is stored in the `pull-request` output variable.
Comment on lines +77 to +78
Copy link
Owner

@Logerfo Logerfo Nov 8, 2019

Choose a reason for hiding this comment

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

Please don't break markdown lines like that. If you're bothered by the horizontal scrollbar, consider activating the word wrap in your text editor (ALT+Z, if you're using VSCode 🙂).

Copy link
Contributor Author

@teleological teleological Nov 8, 2019

Choose a reason for hiding this comment

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

My editor did this without me noticing. Will correct.


### Auto update
You can use (at your own risk) the `release` branch instead of the specific version tag.
Never user `master`, since the distribution file does not exist in this branch and the action will always fail.
Expand Down
1 change: 1 addition & 0 deletions src/main.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,7 @@ async function push() {
core.info(`Label ${label} added to #${pull_number}.`);
core.debug(JSON.stringify(labelsResponse.data));
}
core.setOutput("pull-request", pull_number.toString());
Copy link
Owner

@Logerfo Logerfo Nov 8, 2019

Choose a reason for hiding this comment

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

I think we're supposed to add this to the action.yml file. Check out the documentation.

if (auto_merge) {
await merge(pull_number);
}
Expand Down