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

Remove misleading crontab guru suggestion from docs #3781

Merged
merged 2 commits into from
Jun 13, 2024

Conversation

wez
Copy link
Contributor

@wez wez commented Jun 11, 2024

Since the underlying syntax is non-standard, it is very confusing and misleading to suggest using a tool with different syntax!

@qwerty287
Copy link
Contributor

Is there an alternative that supports our format?

@qwerty287 qwerty287 added the documentation docu & docs label Jun 13, 2024
@qwerty287
Copy link
Contributor

Your first link gives me a 404, and the others are not our format I think.

They're the "quartz" format and see https://github.com/robfig/cron?tab=readme-ov-file#background---cron-spec-format, the default for our cron scheduler is standard cron, but just with seconds added.
quartz adds some more things I think, e.g. year. Also, it looks like we don't have the ?.

I found https://www.freetool.dev/crontab-generator which seems to use exactly our format but it has ads and trackers unfortunately.

Or https://bradymholt.github.io/cRonstrue which is open source but it only supports creating a readable string from a cron and not the other way.

@qwerty287
Copy link
Contributor

qwerty287 commented Jun 13, 2024

To be honestly I think it would be the best to disable seconds in the cron completely as it makes it same to the linux standard format. But that would be a breaking change…

EDIT: Just did that in #3785 - let's continue discussion there for this topic.

@zc-devs
Copy link
Contributor

zc-devs commented Jun 13, 2024

Your first link gives me a 404

Yeah, it is working in my general browser (maybe cached), but not working in other or in incognito. I hope it is temporary issue.
Anyway, it is self-hosted it-tools project. The same is deployed at https://it-tools.tech/.

@qwerty287
Copy link
Contributor

Ah, yes thanks works for me there (https://it-tools.tech/crontab-generator). This seems fine too. @wez Can you update the link to it?

wez added 2 commits June 13, 2024 06:12
Since the underlying syntax is non-standard, it is very confusing and misleading to suggest using a tool with different syntax!
@woodpecker-bot
Copy link
Contributor

woodpecker-bot commented Jun 13, 2024

Deployment of preview was successful: https://woodpecker-ci-woodpecker-pr-3781.surge.sh

@qwerty287 qwerty287 merged commit 6a0c236 into woodpecker-ci:main Jun 13, 2024
7 checks passed
@wez wez deleted the patch-1 branch June 13, 2024 14:19
@woodpecker-bot woodpecker-bot mentioned this pull request Jun 13, 2024
1 task
6543 pushed a commit to 6543-forks/woodpecker that referenced this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation docu & docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants