Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Conversation

@mvines
Copy link
Contributor

@mvines mvines commented Mar 3, 2020

Sometimes you want --confirmed, sometimes no.

@mvines mvines added v0.23 and removed v0.23 labels Mar 3, 2020
@codecov
Copy link

codecov bot commented Mar 3, 2020

Codecov Report

Merging #8597 into master will decrease coverage by 0.0%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master   #8597     +/-   ##
========================================
- Coverage    80.0%   80.0%   -0.1%     
========================================
  Files         256     256             
  Lines       55727   55746     +19     
========================================
- Hits        44609   44608      -1     
- Misses      11118   11138     +20     

@garious
Copy link
Contributor

garious commented Mar 4, 2020

A more verbose PR description would be nice.

@garious
Copy link
Contributor

garious commented Mar 4, 2020

Also confused about the use of "commitment" in the title and "confirmed" elsewhere

@mvines
Copy link
Contributor Author

mvines commented Mar 4, 2020

I'm just using, and remaining consistent with the terminology in the code. Makes for an easier search and replace later

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.

2 participants