Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Clean formulas command #288

Merged

Conversation

henriquemoraes8
Copy link
Contributor

- What I did

  • added rit clean formulas command that cleans ~/.rit/formulas folder

  • Created a utilitary method in stdin to overwrite the os.Stdin and allow for stdin tests

  • Proposed a stdin test for increased command coverage

- How to verify it
run rit clean formulas and verify your saved formulas

- Description for the changelog
added rit clean formulas command to cleanup downloaded formulas

@codecov-commenter
Copy link

codecov-commenter commented Jul 16, 2020

Codecov Report

Merging #288 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #288   +/-   ##
=======================================
  Coverage   69.34%   69.34%           
=======================================
  Files          73       73           
  Lines        2799     2799           
=======================================
  Hits         1941     1941           
  Misses        624      624           
  Partials      234      234           

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 f474ad7...550f255. Read the comment docs.

@sandokandias sandokandias changed the title [FEATURE] rit clean formulas Clean formulas command Jul 17, 2020
@sandokandias sandokandias added the ✨ feature Suggest a new feature or enhancement to the Ritchie project label Jul 17, 2020
pkg/cmd/clean.go Outdated Show resolved Hide resolved
@viniciusramosdefaria viniciusramosdefaria merged commit 9d688e8 into ZupIT:master Jul 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
✨ feature Suggest a new feature or enhancement to the Ritchie project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants