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

Remove flux-up and flux-topo #960

Merged
merged 6 commits into from Jan 19, 2017

Conversation

Projects
None yet
4 participants
@chu11
Copy link
Contributor

chu11 commented Jan 19, 2017

Per discussion in #763

chu11 added some commits Jan 19, 2017

cmd/flux-up: Remove subcommand
Remove subcommand as 'live' module is no longer loaded by default.

Reconsider for future as resiliency is added into Flux.
cmd/flux-topo: Remove subcommand
Remove subcommand as 'live' module is no longer loaded by default.

Reconsider for future as resiliency is added into Flux.

Fixes #763
@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 19, 2017

Coverage Status

Coverage increased (+0.07%) to 76.257% when pulling 8c562d1 on chu11:issue763 into e2528d1 on flux-framework:master.

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Jan 19, 2017

Current coverage is 75.98% (diff: 100%)

Merging #960 into master will increase coverage by 0.07%

@@             master       #960   diff @@
==========================================
  Files           153        152     -1   
  Lines         26030      25923   -107   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits          19760      19697    -63   
+ Misses         6270       6226    -44   
  Partials          0          0          

Powered by Codecov. Last update e2528d1...8c562d1

@garlick

This comment has been minimized.

Copy link
Member

garlick commented Jan 19, 2017

Thanks! Merging.

@garlick garlick merged commit 7332a0a into flux-framework:master Jan 19, 2017

4 checks passed

codecov/patch Coverage not affected when comparing e2528d1...8c562d1
Details
codecov/project 75.98% (+0.07%) compared to e2528d1
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.07%) to 76.257%
Details

@grondo grondo referenced this pull request Mar 28, 2017

Closed

0.7.0 Release Notes #1019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.