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

Jump to bottom

Transfer changes from beta version to main #709

Merged
merged 133 commits into from Dec 4, 2023
Merged

Transfer changes from beta version to main #709

merged 133 commits into from Dec 4, 2023

Conversation

MaksimZhukov
Copy link
Contributor

Description:
This pull request contains the following breaking changes:

  1. The ability to apply labels based on the names of base and/or head branches.
  2. The match object for changed files was expanded with new combinations in order to make it more intuitive and flexible.
  3. The bug related to the sync-labels input was fixed.
  4. By default, dot input is set to true.
  5. Node.js runtime updated to 20.

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

joshdales and others added 30 commits August 7, 2021 15:06
@joshdales
Add branch to MatchConfig interface
2bf42e4
@joshdales
Add function for checking branches
ad73546
@joshdales
include checkBranch in checkMatch
6c50d09
@joshdales
Add a new fixture and test for the branch checking
ee0e0eb
@joshdales
Add another test to make sure that partial branch naming works
a01b9ae
@joshdales
Update new test description
bce88a9
@joshdales
Run build
765934f
@amiel
Fix PR branch labeler
827e118
@amiel
Run build
8aa7614
@amiel
Use correct branch name and update tests
cb5f448
@amiel
Run build
2ced1f3
@amiel
Format
d9ed3e8
@amiel
Include a test for branching
79c0cc7
@amiel @joshdales
Update src/labeler.ts
27a1d89 
Safer access of pull_request data.

Co-authored-by: Josh Dales <30500472+joshdales@users.noreply.github.com>
@joshdales
Merge pull request #1 from bentohq/fix-branch-labeler
7624214
@joshdales
Allow branch config to be an array as well
2d63815
@joshdales
Add tests for array branh labelling
ab49f7a
@joshdales
Run build
2246b66
@joshdales
minor adjustment for successful branch matching.
89f6b77 
To better follow the any convention as opposed to all
@joshdales
Update the tests for applying multiple branch based labels
7aadc17
@joshdales
run the build script
71fc664
@joshdales
Merge branch 'main' into main
818399d
@joshdales
Update README with reference to branch options
6e27606
@joshdales
Merge branch 'main' into main
0ad789c
@joshdales
Run the build command
0861fa5
@joshdales
Fix typo in the README
4c74e84
@joshdales
Merge branch 'main' into main
7f8d8e4
@joshdales
Run prettier
c54c5a2
@joshdales
Run the build command
7b1327b
@joshdales
Rename the getBranchName helper
8c59ecc
joshdales and others added 23 commits May 5, 2023 09:05
@joshdales
Consolidate the new any change files test into the old one
9cfddd0
@joshdales @MaksimZhukov
Update text in test descriptions and logging
51cc5e0 
Co-authored-by: MaksimZhukov <46996400+MaksimZhukov@users.noreply.github.com>
@joshdales
Move the allowed Matchconfig keys into a constant
09645fd
@joshdales
Update the validation when there are no options in the matchConfigs
34a5bf6
@joshdales
Add a guard clause to stop false changed-files positives
4ac1764
@joshdales
Run the build command
a5bed11
@joshdales
Add check for empty objects in checkAll
a256a58
@joshdales
Better check for empty configs in checkAll
57d3407
@joshdales
Bring test I accidently deleted
3352df1
@MaksimZhukov
Merge pull request #203 from joshdales/main
e1fcf6f 
Assigns labels based on branch names
@MaksimZhukov
fix: correct reading of sync-labels input
26342ba
@MaksimZhukov
Merge pull request #574 from MaksimZhukov/v5.0.0-beta.1
4f05277 
fix: correct reading of sync-labels input
@MaksimZhukov
[Beta] Implement the new structure of the match object for the change…
f4eefdc 
…d-files section (#680)

* Implement the new structure of the match object for changed files section

* Replace inner loops with the find method
@MaksimZhukov
Merge branch 'main'
917e5cc
@MaksimZhukov
Change version of Node.js runtime to node20
ab7411e
@MaksimZhukov
5.0.0
1375c42
@MaksimZhukov
Update documentation
d0d0bbe
@MaksimZhukov
Update Node.js version in reusable workflows
8dc8d18
@MaksimZhukov
Run Prettier
c0957ad
@MaksimZhukov
Merge pull request #699 from MaksimZhukov/update-node-runtime-and-dep…
efe4c1c 
…endencies

Update Node.js runtime and dependencies
@MaksimZhukov
Apply suggestions for the beta vesrion and update the documentation
9181355
@MaksimZhukov
Update dist
cb66c2f
@MaksimZhukov
Merge pull request #700 from MaksimZhukov/apply-suggestions-and-updat…
000ca75 
…e-documentation

Apply suggestions for the beta version and update the documentation
@MaksimZhukov MaksimZhukov requested a review from a team as a code owner December 4, 2023 12:43
@MaksimZhukov MaksimZhukov merged commit 8558fd7 into main Dec 4, 2023
7 checks passed
bgu5678
bgu5678 reviewed Jan 3, 2024
View reviewed changes
Copy link

@bgu5678 bgu5678 left a comment

Choose a reason for hiding this comment

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

更新

bgu5678
bgu5678 reviewed Jan 3, 2024
View reviewed changes
Copy link

@bgu5678 bgu5678 left a comment

Choose a reason for hiding this comment

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

更新刪除

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bgu5678 bgu5678 bgu5678 left review comments

@dmitry-shibanov dmitry-shibanov dmitry-shibanov approved these changes

@IvanZosimov IvanZosimov IvanZosimov approved these changes

@nikolai-laevskii nikolai-laevskii nikolai-laevskii approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@MaksimZhukov @dmitry-shibanov @IvanZosimov @nikolai-laevskii @bgu5678 @joshdales @amiel