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.

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

refactor: use click #534

Conversation

sileht
Copy link
Member

@sileht sileht commented Nov 1, 2024

This introduces a CLI interface breakage, due to how click work

This does not work anymore

$ mergify stack push --token foobar

Token (and other global args) are to be passed like this:

$ mergify --token foobar stack push

@sileht
Copy link
Member Author

sileht commented Nov 1, 2024

This pull request is part of a stack:

  1. refactor: use click (#534) 👈
  2. chore(tests): split test_mergify_cli (#535)
  3. fix: correctly pass the repository to /seeach/issues API (#536)

Copy link
Contributor

mergify bot commented Nov 1, 2024

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Changelog requirements

Wonderful, this rule succeeded.
  • any of:
    • -title ~= ^feat
    • label = need changelog
    • label = skip changelog

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

@sileht sileht force-pushed the devs/sileht/merged-handling/Ic4dee15f244a08330482808942e0e0d9a9901dc6 branch from 8d40edd to 741d703 Compare November 1, 2024 22:58
@sileht sileht force-pushed the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch from dfc548c to 2bd3725 Compare November 1, 2024 22:58
@mergify mergify bot requested a review from a team November 1, 2024 23:00
@sileht sileht force-pushed the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch from 2bd3725 to 1bdf3db Compare November 1, 2024 23:11
@Greesb
Copy link
Member

Greesb commented Nov 2, 2024

What's the advantage of click over argparse ?

Syffe
Syffe previously approved these changes Nov 4, 2024
@sileht sileht force-pushed the devs/sileht/merged-handling/Ic4dee15f244a08330482808942e0e0d9a9901dc6 branch from 741d703 to fa84d92 Compare November 4, 2024 15:12
@sileht sileht force-pushed the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch from 1bdf3db to 7b9eb33 Compare November 4, 2024 15:12
@mergify mergify bot dismissed Syffe’s stale review November 4, 2024 15:12

Pull request has been modified.

@mergify mergify bot requested a review from a team November 4, 2024 15:13
@sileht sileht force-pushed the devs/sileht/merged-handling/Ic4dee15f244a08330482808942e0e0d9a9901dc6 branch from fa84d92 to 6bd1a05 Compare November 4, 2024 17:01
@sileht sileht force-pushed the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch from 7b9eb33 to 3592925 Compare November 4, 2024 17:01
@sileht sileht force-pushed the devs/sileht/merged-handling/Ic4dee15f244a08330482808942e0e0d9a9901dc6 branch from 6bd1a05 to ec29a85 Compare November 5, 2024 14:41
@sileht sileht force-pushed the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch from 3592925 to c058501 Compare November 5, 2024 14:41
@sileht sileht force-pushed the devs/sileht/merged-handling/Ic4dee15f244a08330482808942e0e0d9a9901dc6 branch from ec29a85 to 9062980 Compare November 6, 2024 15:17
@sileht sileht force-pushed the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch from c058501 to 2a0f4fe Compare November 6, 2024 15:17
Syffe
Syffe previously approved these changes Nov 8, 2024
@sileht sileht force-pushed the devs/sileht/merged-handling/Ic4dee15f244a08330482808942e0e0d9a9901dc6 branch from 9062980 to efd00cc Compare November 8, 2024 19:28
@sileht sileht force-pushed the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch from 2a0f4fe to e4da612 Compare November 8, 2024 19:28
@mergify mergify bot dismissed Syffe’s stale review November 8, 2024 19:29

Pull request has been modified.

@mergify mergify bot requested a review from a team November 8, 2024 19:29
Base automatically changed from devs/sileht/merged-handling/Ic4dee15f244a08330482808942e0e0d9a9901dc6 to main November 13, 2024 08:51
Copy link
Contributor

mergify bot commented Nov 13, 2024

@sileht this pull request is now in conflict 😩

@mergify mergify bot added the conflict label Nov 13, 2024
lecrepont01
lecrepont01 previously approved these changes Nov 13, 2024
@sileht sileht force-pushed the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch from e4da612 to 48b73d2 Compare November 13, 2024 10:02
@mergify mergify bot removed the conflict label Nov 13, 2024
@mergify mergify bot dismissed lecrepont01’s stale review November 13, 2024 10:03

Pull request has been modified.

This introduces a CLI interface breakage, due to how click work

This does not work anymore

$ mergify stack push --token foobar

Token (and other global args) are to be passed like this:

$ mergify --token foobar stack push

Change-Id: I0819ea54489367dcfeb52a70c5257edc7d3d5c14
@sileht sileht force-pushed the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch from 48b73d2 to f5256b1 Compare November 13, 2024 10:05
@mergify mergify bot requested a review from a team November 13, 2024 10:06
@mergify mergify bot merged commit 5184ed0 into main Nov 14, 2024
5 checks passed
@mergify mergify bot added the queued label Nov 14, 2024
@mergify mergify bot deleted the devs/sileht/merged-handling/I0819ea54489367dcfeb52a70c5257edc7d3d5c14 branch November 14, 2024 09:22
@mergify mergify bot removed the queued label Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants