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

feat: Zenodo remote provider for transparent storage on and retrieval from Zenodo #1455

Merged
merged 2 commits into from Mar 4, 2022

Conversation

johanneskoester
Copy link
Contributor

@johanneskoester johanneskoester commented Mar 3, 2022

Follow-up on #125

tpall and others added 2 commits Mar 3, 2022
… from Zenodo (#125)

* added zenodo remote provider

* added test

* updated formatting

* added sandbox, access_token is required, create new deposition when deposition id is not provided

* updated test Snakefile

* added initial zenodo remote documentation

* added zenodo branches for ci

* fixed code formatting

* added zenodo sandbox pat to workflow

* added zenodo remote test

* fixed formatting

* deleted comment

* skip tibanna test for now

* deleted branch

* added zenodo file exception when hits api limit

* moves file size assertion to remoteobject

* updated test

* added test file

* run also too large upload test

* updated docs

* Delete superfluous file.

* removed error handling around requests and re as unnecessary

* updated error message

* moved download code to remoteobject

* use download url

* fixed formatting

* upload large files

* fixed formatting

* removed makedir

* updated test, download to subdir

* moved exception handling to request wrapper

* removed zen branch from ci

* fixed deposition id handling

* fixed stats is not defined

* fix download

* fixed test download path, updated docs

* Only run test if token is available.

* fmt

* envvars

* fix skip criterion

* fix error handling

* windows secret

Co-authored-by: Johannes Köster <johannes.koester@uni-due.de>
Co-authored-by: Johannes Köster <johannes.koester@tu-dortmund.de>
@sonarcloud
Copy link

sonarcloud bot commented Mar 4, 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 4586ef7 into main Mar 4, 2022
7 checks passed
@johanneskoester johanneskoester deleted the zenodo branch Mar 4, 2022
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