Skip to content

Conversation

@leocnp
Copy link
Contributor

@leocnp leocnp commented Dec 2, 2024

Global options --token and --github-server are related to the stack group and should not have their defaults computed for other groups since they may break others commands.

--debug is still a global option:

$ mergify --debug stack push

This does not work anymore:

$ mergify --token foobar stack push

Global stack options are to be passed like this:

$ mergify stack --token foobar push

@mergify
Copy link
Contributor

mergify bot commented Dec 2, 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

@leocnp leocnp requested a review from a team December 3, 2024 08:53
Those options are related to the `stack` group and should not be computed
for other groups since they may break others calls.

--debug is still a global option:

$ mergify --debug stack push

This does not work anymore:

$ mergify --token foobar stack push

Global stack options are to be passed like this:

$ mergify stack --token foobar push
@leocnp leocnp force-pushed the fix-default-options branch from 743d4ae to a289198 Compare December 3, 2024 08:55
mergify bot added a commit that referenced this pull request Dec 3, 2024
@mergify mergify bot added the queued label Dec 3, 2024
@mergify mergify bot merged commit aaa1d7f into Mergifyio:main Dec 3, 2024
5 checks passed
@mergify mergify bot removed the queued label Dec 3, 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.

2 participants