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

Jkmarx/update management command argparse #2607

Merged

Conversation

jkmarx
Copy link
Member

@jkmarx jkmarx commented Feb 20, 2018

No description provided.

@codecov
Copy link

codecov bot commented Feb 20, 2018

Codecov Report

Merging #2607 into jkmarx/django-upgrade-1.8 will increase coverage by 0.01%.
The diff coverage is 83.33%.

Impacted file tree graph

@@                      Coverage Diff                      @@
##           jkmarx/django-upgrade-1.8    #2607      +/-   ##
=============================================================
+ Coverage                      55.02%   55.04%   +0.01%     
=============================================================
  Files                            400      400              
  Lines                          25750    25694      -56     
  Branches                        1240     1240              
=============================================================
- Hits                           14170    14143      -27     
+ Misses                         11580    11551      -29
Impacted Files Coverage Δ
refinery/galaxy_connector/tests.py 100% <100%> (ø) ⬆️
...inery/core/management/commands/set_up_site_name.py 91.3% <100%> (-1.8%) ⬇️
refinery/core/tests.py 99.5% <100%> (ø) ⬆️
...ctor/management/commands/create_galaxy_instance.py 69.44% <53.84%> (+22.27%) ⬆️
.../core/management/commands/create_workflowengine.py 89.65% <85.71%> (+26.24%) ⬆️

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 24fa9f4...2cacc6e. Read the comment docs.

Copy link
Member

@scottx611x scottx611x left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, lets see a passing build from the tests I just added before a merge

@scottx611x scottx611x merged commit 6842262 into jkmarx/django-upgrade-1.8 Feb 20, 2018
@scottx611x scottx611x deleted the jkmarx/update-management-command-argparse branch February 20, 2018 20:08
scottx611x pushed a commit that referenced this pull request Feb 21, 2018
* Upgrade django version to 1.8

* Update workflow_manager httpReponse to jsonReponse.

* Add allowed_hosts to dev setting.

* Update comment.

* Remove conflicting settings, model can only have one of add, add_on, or default.

* Update analysis_manager httpReponse to jsonResponse.

* Add migrations.

* Update module_name with model name.

* Refactor.

* Upgrade some dependencies for django 1.8

* Revert to minimum dependency upgrade to resolve migration issue.

* Update annotation server with jsonResponse.

* Update core and data_set_manager apps.

* Fix typo.

* Update requirements to fix migration issue.

* Try release which seperates dependency on User module.

* Check to see if chunked-upload is the problem.

* Fix bug related to abstract model causing migration to fail for library.

* Remove duplicate setting.

* Track migration until library updates

* Reorder installed apps per redux docs.

* Update activate_user with parser.

* Fix pep8 in migration.

* Remove unit test due to inaccurate scope.

* Add mocks to unit test.

* Remove unneccessary deletion.

* Fix selium unit test errors.

* Remove accidently added json

* Jkmarx/registration redux upgrade (#2562)

* update redux.

* Update method params changed from upgrade

* Update flockblock params due to upgrade.

* Refactor, return_value to inline.

* Fix unit test.

* Fix mistake using jsonresponse with a str.

* Remove typo.

* Remove unneeded field.

* Chunked_upload migration.

* Opt for safer option for jsonResponse.

* Fix syntax.

* Fix pep8.

* Swap deprecated method out. (#2566)

* Swap deprecated method out.

* Remove solr_igv dependencies. (#2570)

* Remove solr_igv dependencies.

* Remove visualization_manager.

* Upgrade django storage. (#2571)

* Upgrade django storage.

* Remove post-css library, unused. (#2574)

* Update user/file facet/columns per user request. (#2577)

* Remove post-css library, unused.

* Update user/file facet/columns per user request.

* Remove typo ,

* Fix typo, double quotes.

* Remove space.

* Add cell_type to facets.

* Fix unit test.

* Add missing facets.

* Fix cypress test.

* Revert to 1.6.0 due to django 1.9 support.

* Remove unnessary flake8 ignore.

* Add json.load exception.

* Jkmarx/update httpresponse (#2533)

* Update workflow_manager httpReponse to jsonReponse.

* Add allowed_hosts to dev setting.

* Update comment.

* Remove conflicting settings, model can only have one of add, add_on, or default.

* Update analysis_manager httpReponse to jsonResponse.

* Add migrations.

* Update module_name with model name.

* Refactor.

* Upgrade some dependencies for django 1.8

* Revert to minimum dependency upgrade to resolve migration issue.

* Update annotation server with jsonResponse.

* Update core and data_set_manager apps.

* Fix typo.

* Update requirements to fix migration issue.

* Try release which seperates dependency on User module.

* Check to see if chunked-upload is the problem.

* Fix bug related to abstract model causing migration to fail for library.

* Remove duplicate setting.

* Track migration until library updates

* Reorder installed apps per redux docs.

* Update activate_user with parser.

* Fix pep8 in migration.

* Remove unit test due to inaccurate scope.

* Add mocks to unit test.

* Remove unneccessary deletion.

* Fix selium unit test errors.

* Remove accidently added json

* Jkmarx/registration redux upgrade (#2562)

* update redux.

* Update method params changed from upgrade

* Update flockblock params due to upgrade.

* Refactor, return_value to inline.

* Fix unit test.

* Fix mistake using jsonresponse with a str.

* Remove typo.

* Remove unneeded field.

* Chunked_upload migration.

* Opt for safer option for jsonResponse.

* Fix syntax.

* Fix pep8.

* Swap deprecated method out. (#2566)

* Swap deprecated method out.

* Remove solr_igv dependencies. (#2570)

* Remove solr_igv dependencies.

* Remove visualization_manager.

* Upgrade django storage. (#2571)

* Upgrade django storage.

* Remove post-css library, unused. (#2574)

* Update user/file facet/columns per user request. (#2577)

* Remove post-css library, unused.

* Update user/file facet/columns per user request.

* Remove typo ,

* Fix typo, double quotes.

* Remove space.

* Add cell_type to facets.

* Fix unit test.

* Add missing facets.

* Fix cypress test.

* Revert to 1.6.0 due to django 1.9 support.

* Remove unnessary flake8 ignore.

* Add json.load exception.

* Jkmarx/remove unused m commands (#2583)

* Remove unused management commands.

* Remove unused method.

* Jkmarx/outparse to argparse (#2584)

* Update make_options to add_arguments for optparse/argparse upgrade.

* Refactor for easier read.

* Jkmarx update deprecated 1 8 items (#2588)

* Update config with the new 1.8 setting.

* Add comment.

* Remove unused template.

* Update template related settings.

* Update get_template per docs, Context&RequestContext unneeded.

* Fix typo in context_processors location update.

* Remove duplicate.

* Fix added setting.

* Add `--fake-initial` to our migrate commands to provide the behavior that Django 1.7 gave by default (#2585)

* Update management command, argparse.

* Scottx611x/fix management commands (#2606)

* Update `set_uo_site_name` to use add_arguments()

* Add test coverage

* Add test for `add_users_to_public_group`

* Add test for user activation

* Add test for `import_annotations`

* Move for consistency.

* Jkmarx/update management command argparse (#2607)

* Update management command, argparse.

* Move for consistency.

* Add tests for `create_workflow_engine`

* Add tests for `create_galaxy_instance`

* Fix bug related to non-obj.
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