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.

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

Initial PR #2

Merged
merged 21 commits into from
Jul 11, 2017
Merged

Initial PR #2

merged 21 commits into from
Jul 11, 2017

Conversation

krallin
Copy link
Collaborator

@krallin krallin commented Jul 11, 2017 •
edited
Loading

Still a bit of work to do here:

  • Respond to SIGTERM / SIGINT
  • Unit tests for cron/cron.go:StartJob (dependent on a having a way to signal it to stop)
  • A handful of integration tests to cover main.go
  • Flag handling
  • Warn on USER declaration (we ignore it)
  • README
  • LICENSE
  • CI
  • Static builds people can download

@krallin krallin force-pushed the master branch 2 times, most recently from bdb82fb to 631bf0e Compare July 11, 2017 10:52
krallin added 19 commits July 11, 2017 12:56
@krallin krallin force-pushed the master branch 9 times, most recently from f22f593 to 063f2f5 Compare July 11, 2017 11:20
@krallin krallin force-pushed the master branch 3 times, most recently from 90ef9e2 to 9a00ee3 Compare July 11, 2017 11:32
@krallin krallin merged commit 640f705 into aptible:master Jul 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@krallin