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

upgrade github.com/gorhill/cronexpr lib #2351

Merged
merged 13 commits into from Aug 4, 2020

Conversation

Dr-wgy
Copy link

@Dr-wgy Dr-wgy commented Jun 10, 2020

Required for all non-trivial PRs
  • Rebased/mergable
  • Tests pass
  • CHANGELOG.md updated
  • Sign CLA (if not already signed)
Required only if applicable

You can erase any checkboxes below this note if they are not applicable to your Pull Request.

@Dr-wgy
Copy link
Author

Dr-wgy commented Jun 10, 2020

upgrade github.com/gorhill/cronexpr version

@Dr-wgy
Copy link
Author

Dr-wgy commented Jun 10, 2020

fix bug #2350

@Dr-wgy
Copy link
Author

Dr-wgy commented Jun 10, 2020

this issue lead kapacitor can not start up

@docmerlin
Copy link
Contributor

Hey @wuguanyu thanks for this PR, but for us to accept this change you would need to update the Gopkg.toml and Gopkg.lock. I can do this for you, or you can if you want?

@Dr-wgy
Copy link
Author

Dr-wgy commented Jul 15, 2020

Hey @wuguanyu thanks for this PR, but for us to accept this change you would need to update the Gopkg.toml and Gopkg.lock. I can do this for you, or you can if you want?

Sorry for that,Can you help me update the Gopkg.toml and Gopkg.lock,Thanks a lot, I will change Gopkg.toml and Gopkg.lock next time

@docmerlin
Copy link
Contributor

Will do!

@docmerlin docmerlin self-assigned this Aug 3, 2020
@docmerlin docmerlin merged commit 8adfc77 into influxdata:master Aug 4, 2020
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.

None yet

4 participants