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

Gap detector repeat #2988

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ethanmdavidson
Copy link

@ethanmdavidson ethanmdavidson commented May 12, 2021

closes #2813

degan6 did all the work in #2872 but it looks like that PR got abandoned for some reason. All I've done here is take that branch and fix the indentation.

@smithjoelt
Copy link

It's been a while since anyone has updated this. Is this issue still alive?

@ethanmdavidson
Copy link
Author

Yes, I would still like to get this merged. This PR hasn't been updated in a while because it hasn't been reviewed, so no changes have been requested.

@smithjoelt
Copy link

Maybe we could ping someone with write access to get their attention.

@smithjoelt
Copy link

Is there a reason we're still using Travis CI on this? I think I remember seeing somewhere that Circle CI is the way to go nowadays. Would this pass Circle CI?

Mind you, I'm just trying to get this issue handled and closed. I'm doing triage on this repo.

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.

Create option for gap detector to create event on every run when gap is detected
3 participants