Skip to content

Release candidate #2298

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

Merged
merged 120 commits into from
Sep 19, 2017
Merged

Release candidate #2298

merged 120 commits into from
Sep 19, 2017

Conversation

josenavas
Copy link
Contributor

No description provided.

josenavas and others added 30 commits August 23, 2017 07:05
[DO NOT MERGE] August 24th Release
replace printable for try/except utf-8
* Adding branch description

* Adding spaces

* Adding timeline
josenavas and others added 27 commits September 15, 2017 11:11
redbiom now adds per sample studies to analysis
* Adding patch to add the 'name' parameter to all validate commands

* Adding the parameter 'name' automatically and adding a test for it

* Removing extra blank line

* Adding dflt value to artifact name

* Edited the wrong file

* Adding user defined name at creation time

* Fixing test
* add is_from_analysis to artifact_handlers

* fix test_post_metadata

* fix error
* fix #1293

* flake8

* fix errors

* addressing @ElDeveloper comments
* Fix #2276

* Factoring out generate nginx directory file list

* Factoring out the nginx file list writing

* Factoring out generating the file list of an artifact

* Factoring out the header setting

* Addressing @antgonza's comment

* Addressing @wasade's comments
@antgonza antgonza merged commit 1e40145 into release-candidate Sep 19, 2017
@codecov-io
Copy link

codecov-io commented Sep 19, 2017

Codecov Report

Merging #2298 into release-candidate will increase coverage by 0.94%.
The diff coverage is 89.49%.

Impacted file tree graph

@@                  Coverage Diff                  @@
##           release-candidate    #2298      +/-   ##
=====================================================
+ Coverage              92.81%   93.76%   +0.94%     
=====================================================
  Files                    171      163       -8     
  Lines                  18925    18550     -375     
=====================================================
- Hits                   17566    17393     -173     
+ Misses                  1359     1157     -202
Impacted Files Coverage Δ
qiita_db/__init__.py 100% <ø> (ø) ⬆️
qiita_db/test/test_artifact.py 99.84% <ø> (ø) ⬆️
qiita_db/commands.py 96.55% <ø> (-0.48%) ⬇️
...ita_pet/handlers/study_handlers/tests/test_base.py 98.14% <ø> (-0.22%) ⬇️
qiita_db/test/test_commands.py 99.6% <ø> (+0.28%) ⬆️
qiita_pet/test/test_prep_template.py 97.87% <ø> (ø) ⬆️
qiita_db/handlers/tests/test_oauth2.py 99.34% <100%> (ø) ⬆️
qiita_pet/test/tornado_test_base.py 91.22% <100%> (+0.31%) ⬆️
qiita_db/test/test_study.py 99.76% <100%> (ø) ⬆️
qiita_db/handlers/processing_job.py 96.92% <100%> (+7.06%) ⬆️
... and 57 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7941067...064d35a. Read the comment docs.

antgonza pushed a commit that referenced this pull request Sep 27, 2017
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.

5 participants