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

Added option to disable hooks on job. I need this, since the class I'… #943

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

Conversation

holgersindbaek
Copy link

…m delaying also have a 'before' method

@coveralls
Copy link

coveralls commented Sep 22, 2016

Coverage Status

Coverage increased (+0.02%) to 91.143% when pulling caa82a7 on holgersindbaek:master into e3772d4 on collectiveidea:master.

1 similar comment
@coveralls
Copy link

coveralls commented Sep 22, 2016

Coverage Status

Coverage increased (+0.02%) to 91.143% when pulling caa82a7 on holgersindbaek:master into e3772d4 on collectiveidea:master.

@albus522
Copy link
Member

I think there might be a better, less all or nothing, approach but this is an interesting issue. We might also need to rely on a less collision prone implementation in future versions but that would need to be phased in to avoid breaking existing apps.

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

3 participants