Skip to content

fix: Ignore boolean flags or args without explicit = sign in cron monitoring #44

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
Merged
stephanschuler merged 1 commit into from
Jul 7, 2025
Merged

fix: Ignore boolean flags or args without explicit = sign in cron monitoring #44

stephanschuler merged 1 commit into from
Jul 7, 2025
+9 −4

Conversation

@paxuclus
Copy link
Member

@paxuclus paxuclus commented Jul 7, 2025 •
edited
Loading

Args may be passed without explicit equal sign, e.g.:

./flow scheduler:pollforincomingjobs --groupName default

Those args would break the aspect, causing flow to throw an exception

@paxuclus paxuclus self-assigned this Jul 7, 2025
@paxuclus paxuclus changed the title fix: Ignore boolean flags or args without explicit = sign in cron mon… fix: Ignore boolean flags or args without explicit = sign in cron mon itoring Jul 7, 2025
@paxuclus paxuclus changed the title fix: Ignore boolean flags or args without explicit = sign in cron mon itoring fix: Ignore boolean flags or args without explicit = sign in cron monitoring Jul 7, 2025
@stephanschuler stephanschuler merged commit 33f3936 into master Jul 7, 2025
24 checks passed
@paxuclus paxuclus deleted the fix/cron-monitoring-bool-flags branch August 20, 2025 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stephanschuler stephanschuler stephanschuler approved these changes

@saschanowak saschanowak saschanowak approved these changes

@bastiBepunkt bastiBepunkt Awaiting requested review from bastiBepunkt

Assignees

@paxuclus paxuclus

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@paxuclus @stephanschuler @saschanowak