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

Add support for IntelliJ cron tab feature #115

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DRSchlaubi
Copy link

No description provided.

@InsanusMokrassar
Copy link
Owner

Hi and thank you for your PR :) is it ready for review?

@DRSchlaubi
Copy link
Author

yeah should be

@InsanusMokrassar
Copy link
Owner

I've checked the Cron Descriptor and looks like it have no any opportunity to configure tooltips for crons. Due to the different formats (in original cron it starts from minutes when krontab starts with seconds), it will be incompatible with krontab library.

If I missunderstood something, feel free to write it :) thank you for your PR anyway

@DRSchlaubi
Copy link
Author

I've checked the Cron Descriptor and looks like it have no any opportunity to configure tooltips for crons. Due to the different formats (in original cron it starts from minutes when krontab starts with seconds), it will be incompatible with krontab library.

If I missunderstood something, feel free to write it :) thank you for your PR anyway

I am talking about this official feature, so you need to use IntelliJ EAP to test this

@InsanusMokrassar
Copy link
Owner

I've checked the Cron Descriptor and looks like it have no any opportunity to configure tooltips for crons. Due to the different formats (in original cron it starts from minutes when krontab starts with seconds), it will be incompatible with krontab library.
If I missunderstood something, feel free to write it :) thank you for your PR anyway

I am talking about this official feature, so you need to use IntelliJ EAP to test this

Got it and will try asap :) thank you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants