Skip to content

Conversation

@Kami
Copy link
Member

@Kami Kami commented Feb 7, 2017

Just some house keeping.

It updates the following dependencies to the latest stable version.

General dependencies

  • paramiko
  • appscheduler
  • pyyaml
  • semver
  • retrying
  • kombu
  • amqp
  • networkx
  • jsonschema
  • sseclient
  • jsonpath-rw

Dev / test dependencies

  • pylint, astroid (done in a separate PR which fixes some lint issues)
  • flake8 (done in a separate PR which fixes some lint issues)
  • mock (done in a separate PR which fixes some lint issues)

I verified the checked and all the changes are backward compatible, but I still think it's better to merge this post v2.2.0.

@codecov-io
Copy link

codecov-io commented Feb 7, 2017

Codecov Report

Merging #3189 into master will decrease coverage by -0.05%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##           master    #3189      +/-   ##
==========================================
- Coverage   77.83%   77.78%   -0.05%     
==========================================
  Files         433      433              
  Lines       22398    22408      +10     
==========================================
- Hits        17433    17429       -4     
- Misses       4965     4979      +14
Impacted Files Coverage Δ
st2client/st2client/commands/action.py 58.49% <ø> (ø)
st2common/st2common/models/db/marker.py 100% <ø> (ø)
st2api/st2api/controllers/resource.py 88.02% <ø> (+1.24%)
st2common/st2common/content/bootstrap.py 0% <ø> (ø)
st2common/st2common/models/db/rule.py 100% <ø> (ø)
st2common/st2common/models/db/rbac.py 100% <ø> (ø)
st2common/st2common/models/db/sensor.py 100% <ø> (ø)
st2common/st2common/operators.py 84.8% <ø> (ø)
st2common/st2common/models/db/rule_enforcement.py 76.67% <ø> (ø)
st2common/st2common/models/db/action.py 100% <ø> (ø)
... and 32 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 ce0eb95...a6c5376. Read the comment docs.

@arm4b
Copy link
Member

arm4b commented Feb 7, 2017

but I still think it's better to merge this post v2.2.0.

+:100:

@Kami
Copy link
Member Author

Kami commented Jun 30, 2017

This PR is obsolete now, will open a new one.

@Kami Kami closed this Jun 30, 2017
@Kami Kami mentioned this pull request Jun 30, 2017
1 task
@arm4b arm4b deleted the upgrade_dependencies branch June 30, 2017 10:46
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.

4 participants