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.

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

Added ability to pass in an optional PR number as a parameter #349

Merged
merged 28 commits into from
Jul 6, 2023
Merged

Added ability to pass in an optional PR number as a parameter #349

Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
baefe5d
Adding pr-number as an optional parameter
credfeto Apr 19, 2022
5c4971e
Updated README
credfeto Apr 19, 2022
52078ca
Tests on the pr-number parameter
credfeto Apr 19, 2022
290b11b
Added missing | to table
credfeto Jun 21, 2023
ff4b71d
re-built script
credfeto Jun 21, 2023
7e9e70f
Adding support for multiple pr-numbers
credfeto Jun 21, 2023
836757c
excluded .idea
credfeto Jun 21, 2023
b6e7208
Updated readme to reflect that there might be more than one PR
credfeto Jul 3, 2023
07ff7ea
Additional warning
credfeto Jul 3, 2023
bdefaa6
Removed unused
credfeto Jul 3, 2023
13ad946
Reformatted and re-built
credfeto Jul 3, 2023
a23ad07
Corrected message
credfeto Jul 3, 2023
24492d8
Removed required check
credfeto Jul 3, 2023
ea5dd1d
Added (s) to pull request numbers in the description
credfeto Jul 5, 2023
d64108f
Reworded PR-number parameter description
credfeto Jul 5, 2023
26e7680
adding getMultilineInput into the tests
credfeto Jul 3, 2023
c9e03c8
Fixing tests for single pr
credfeto Jul 3, 2023
5b2ecd5
Fixing tests for multiple prs
credfeto Jul 3, 2023
9df982d
Updated README.md to make it more obvious that it can take a list of PRs
credfeto Jul 3, 2023
33d0803
Added example that labels PR's 1-3
credfeto Jul 5, 2023
71ff194
Handled no pull requests better (from code review)
credfeto Jul 5, 2023
8939095
Handled no pull requests better (from code review)
credfeto Jul 5, 2023
4bad0c3
Handled missing pull request better (from code review)
credfeto Jul 5, 2023
5b72df0
Back out suggested change as it broke the tests
credfeto Jul 6, 2023
4973c4e
Rebuilt dist
credfeto Jul 6, 2023
deae017
Update src/labeler.ts
credfeto Jul 6, 2023
2cd40dd
Added Emphasis to the note
credfeto Jul 6, 2023
98b70ef
Changed mockInput for pr-number to be string[]
credfeto Jul 6, 2023
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
Prev Previous commit
Next Next commit
Updated readme to reflect that there might be more than one PR
  • Loading branch information
@credfeto
credfeto committed Jul 3, 2023
commit b6e72084d21cdbbd90b5051f20b34882d0f4ab33
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ Various inputs are defined in [`action.yml`](action.yml) to let you configure th
| `configuration-path` | The path to the label configuration file | `.github/labeler.yml` |
| `sync-labels` | Whether or not to remove labels when matching files are reverted or no longer changed by the PR | `false` |
| `dot` | Whether or not to auto-include paths starting with dot (e.g. `.github`) | `false` |
| `pr-number` | The PR number of a PR that to update, rather than detecting from the workflow context | N/A |
| `pr-number` | The PR number(s) of PRs to update, rather than detecting from the workflow context | N/A |
credfeto marked this conversation as resolved.
Show resolved Hide resolved

When `dot` is disabled and you want to include _all_ files in a folder:

Expand Down