-
Notifications
You must be signed in to change notification settings - Fork 406
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
fix: Limit number of labels added to 100 #497
Merged
Merged
Changes from 9 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
756f0bd
push to excess labels to avoid reaching the limit
markmssd e4e6956
build dist
markmssd 7429fbb
never set more than 100 labels
markmssd e173de7
use splice instead of set
markmssd 56b1a22
Merge branch 'main' into fix/max-100-labels
markmssd f90e3c3
Merge branch 'main' into fix/max-100-labels
markmssd 0e55d23
ignore IDE folders
markmssd 4beee0f
install @octokit/plugin-retry
markmssd 267e6ba
always setLabels
markmssd b35f25a
fix indentations
markmssd 5baeff5
fix specs
markmssd d65a48d
add spec for excess labels
markmssd 4dd2b81
prettier
markmssd 2f9caa0
licensed cache
markmssd 112b189
revert to !!core.getInput('sync-labels')
markmssd 3d7acab
better warning for exceeded labels
markmssd 92a11d8
keep manually-added labels
markmssd ffc8ca7
nest the dedupe logic
markmssd 3c2f134
rename `removeLabel` to `removeLabelFromList` to avoid confusion
markmssd 66f07b8
Merge branch 'main' into fix/max-100-labels
unrealclouder 287ce90
use Sets, and issue a call only if labels have actually changed
unrealclouder 05a4b4b
remove IDE config folders from gitignore
unrealclouder c9ee1b4
remove obsolete duplucation check
unrealclouder File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,5 @@ | ||
.DS_Store | ||
node_modules/ | ||
lib/ | ||
lib/ | ||
.vscode/ | ||
.idea/ | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Till the moment the IDE settings were avoided to be added to the repo
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.
Oh I can remove them haha, it was just getting annoying locally