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

support specify user to run job #109

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

Conversation

w136712058
Copy link

Some times we need specify user to run some job for the reason such as authority problem (such as file I/O authority).I modify it to solve this problem and it work perfectly on my enviroment.

@coveralls
Copy link

coveralls commented Dec 23, 2019

Coverage Status

Coverage decreased (-11.4%) to 78.488% when pulling 907ca02 on w136712058:master into 978a26d on kraiz:master.

@w136712058
Copy link
Author

@coveralls

@w136712058
Copy link
Author

@coveralls Why the check result always has “django_crontab/crontab.py:18:19: F821 undefined name 'basestring'”?
Even changed nothing it also occur?
What problem here:

string_type = basestring if sys.version_info[0] == 2 else str # flake8: noqa

or here:

if sys.version_info[0] == 2:
string_type = basestring
else:
string_type = str

@yapici
Copy link

yapici commented Jul 17, 2020

@w136712058, thank you for this PR. Do you have plans to work on this to address the failed tests?

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