Skip to content
This repository has been archived by the owner on Nov 15, 2021. It is now read-only.

Auto-Delete Bootstrap File #986

Merged
merged 2 commits into from
Jul 17, 2019

Conversation

brianlenz
Copy link
Contributor

What current issue(s) does this address, or what feature is it adding?

Auto-delete the downloaded bootstrap file after running np-bootstrap.

How did you solve this problem?

  • Updated the np-bootstrap process so that it will delete the downloaded bootstrap file by default when it's done.
  • Added a -k/--keep-bootstrap-file flag to np-bootstrap to allow you to keep the bootstrap file in case you need to re-use the file after download.

How did you make sure your solution works?

Tested it with both configurations.

Are there any special changes in the code that we should be aware of?

No.

Please check the following, if applicable:

  • Did you add any tests?
  • Did you run make lint?
  • Did you run make test?
  • Are you making a PR to a feature branch or development rather than master?
  • Did you add an entry to CHANGELOG.rst? (if not, please do)

Brian Lenz added 2 commits July 16, 2019 09:50
* Updated the `np-bootstrap` process so that it will delete the downloaded bootstrap file by default when it's done.
* Added a `-k`/`--keep-bootstrap-file` flag to `np-bootstrap` to allow you to keep the bootstrap file in case you need to re-use the file after download.
* Added the PR link to the changelog.
@coveralls
Copy link

coveralls commented Jul 16, 2019

Coverage Status

Coverage increased (+0.002%) to 85.289% when pulling 671be5b on brianlenz:auto_delete_bootstrap_file into c06df37 on CityOfZion:development.

@ixje
Copy link
Member

ixje commented Jul 17, 2019

👍

@ixje ixje merged commit 171d2a4 into CityOfZion:development Jul 17, 2019
@brianlenz brianlenz deleted the auto_delete_bootstrap_file branch July 17, 2019 11:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants