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

Reduce flake8 max-line-length #3222

Closed
wants to merge 5 commits into from
+0 −0
Closed

Reduce flake8 max-line-length #3222

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
60b4311
Reduce flake8 max-line-length
mirembe-mariam Mar 9, 2024
65d5bb1
Reduce flake8 max-line-length
mirembe-mariam Mar 10, 2024
f2da87e
write_submit_script should return nothing
mirembe-mariam Mar 12, 2024
edeee2a
Merge branch 'Parsl:master' into master
mirembe-mariam Mar 17, 2024
3d8ce40
Merge branch 'master' into master
mirembe-mariam Mar 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Loading
Loading status checks…
write_submit_script should return nothing
@mirembe-mariam
mirembe-mariam committed Mar 12, 2024

Unverified

No user is associated with the committer email.
Learn about vigilant mode
commit f2da87e742a54c92ab7cdfb3bf27c6f53f88fb1b
4 changes: 2 additions & 2 deletions parsl/providers/cluster_provider.py
View file
Original file line number Diff line number Diff line change
@@ -91,7 +91,7 @@ def _write_submit_script(self, template, script_filename, job_name, configs):
- configs (dict) : configs that get pushed into the template

Returns:
- True: on success
- None

Raises:
SchedulerMissingArgs : If template is missing args
@@ -117,7 +117,7 @@ def _write_submit_script(self, template, script_filename, job_name, configs):
logger.error("Uncategorized error: %s", e)
raise e

return True
return

@abstractmethod
def _status(self):