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

fix: None type error when invoking Workflow object manually #1731

Merged
merged 3 commits into from Sep 16, 2022

Conversation

Hoeze
Copy link
Contributor

@Hoeze Hoeze commented Jun 20, 2022

Description

After updating snakemake to its current version, the following code fails with TypeError: 'NoneType' object is not iterable:

workflow = Workflow(snakefile=snakefile)

This PR attempts to fix that issue.

QC

  • The PR contains a test case for the changes or the changes are already covered by an existing test case.
  • The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

@Hoeze Hoeze changed the title Fix None type error when invoking Workflow object manually Fix: None type error when invoking Workflow object manually Jun 20, 2022
@sonarcloud
Copy link

sonarcloud bot commented Jun 20, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Hoeze
Copy link
Contributor Author

Hoeze commented Jul 19, 2022

@johanneskoester could you please review / merge this PR?

@Hoeze
Copy link
Contributor Author

Hoeze commented Aug 9, 2022

@johanneskoester Any updates on this?
This bug is really annoying but also easily fixed by this PR...

@Hoeze Hoeze changed the title Fix: None type error when invoking Workflow object manually fix: None type error when invoking Workflow object manually Aug 9, 2022
@sonarcloud
Copy link

sonarcloud bot commented Aug 26, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@johanneskoester johanneskoester merged commit dc45ccb into snakemake:main Sep 16, 2022
@Hoeze Hoeze deleted the patch-1 branch September 16, 2022 11:57
johanneskoester pushed a commit that referenced this pull request Sep 26, 2022
🤖 I have created a release *beep* *boop*
---


##
[7.14.1](v7.14.0...v7.14.1)
(2022-09-23)


### Bug Fixes

* allocation of local ssds in k8s tests
([#1870](#1870))
([d0de4dc](d0de4dc))
* allow script directive to take pathlib Path
([#1869](#1869))
([12cdc96](12cdc96))
* catch errors in remote.AUTO provider list
([#1834](#1834))
([c613ed2](c613ed2))
* consistently use text output in conda shell commands and various
little fixes for failing test cases due to conda package changes
([#1864](#1864))
([4234fe7](4234fe7))
* declare associative arrays
([#1844](#1844))
([90ae449](90ae449))
* fix falsely triggered reruns if input files are obtained via
workflow.source_path()
([#1862](#1862))
([2dc2e6a](2dc2e6a))
* fixed typos
([#1847](#1847))
([a1e49b6](a1e49b6))
* k8s container volume mounts as list
([#1868](#1868))
([5c54df3](5c54df3))
* None type error when invoking Workflow object manually
([#1731](#1731))
([dc45ccb](dc45ccb))
* request disk_mb resource from k8s
([#1858](#1858))
([f68f166](f68f166))
* respect shebang lines in post-deploy scripts (see deployment docs)
([#1841](#1841))
([c26c4b6](c26c4b6))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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

2 participants