996: Making ListEnvelope public

[piotrkot]

Making ListEnvelope public

[codecov-io]

Codecov Report

Merging #1073 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1073      +/-   ##
============================================
+ Coverage     87.63%   87.63%   +<.01%     
  Complexity     1529     1529              
============================================
  Files           269      270       +1     
  Lines          3929     3931       +2     
  Branches        215      216       +1     
============================================
+ Hits           3443     3445       +2     
  Misses          437      437              
  Partials         49       49

Impacted Files	Coverage Δ	Complexity Δ
src/main/java/org/cactoos/list/ListEnvelope.java	100% <ø> (ø)	12 <0> (ø)	⬇️
src/main/java/org/cactoos/func/Async.java	100% <0%> (ø)	6% <0%> (-3%)	⬇️
...ava/org/cactoos/collection/CollectionEnvelope.java	100% <0%> (ø)	26% <0%> (-1%)	⬇️
src/main/java/org/cactoos/text/TrimmedText.java
src/main/java/org/cactoos/text/JoinedText.java
src/main/java/org/cactoos/text/NormalizedText.java
src/main/java/org/cactoos/text/Normalized.java	100% <0%> (ø)	3% <0%> (?)
src/main/java/org/cactoos/text/Trimmed.java	100% <0%> (ø)	2% <0%> (?)
src/main/java/org/cactoos/scalar/Binary.java	100% <0%> (ø)	4% <0%> (?)
src/main/java/org/cactoos/text/Joined.java	100% <0%> (ø)	6% <0%> (?)

---

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 77aa19a...b56b027. Read the comment docs.

[0crat]

@piotrkot/z this pull request is too small, just 5 lines changed (less than 10), there will be no formal code review, see §53 and §28; in the future, try to make sure your pull requests are not too small; @llorllale/z please review this and merge or reject

[llorllale]

@piotrkot please fix your commit msgs (a47be69 is unacceptable)

[piotrkot]

@llorllale what do you mean? The first two messages were not acceptable. The third one was fine for Travis.

[llorllale]

@piotrkot the message on a47be69 is not acceptable

[llorllale]

@piotrkot ping

[piotrkot]

@llorllale Can you explain what should be in the commit message?
Normally, I put message describing what I actually did in the commit. So, the first one is fixing the issue, The second is fixing qulice warning, and as sole purpose of the third change was to satisfy travis' regex check, that's what is says.
Now, if you don't like it I will make a rollback of the previous commit (a47be69), which will still be a dummy commit. How should I name it?

[llorllale]

@piotrkot you can just squash them all into a single commit with a sensible message

[piotrkot]

@llorllale OMG, I hate git. I couldn't make it with rebase and squash. Instead, tried with git reset --soft HEAD~3 and git commit. Any better?

[piotrkot]

@llorllale ping

[llorllale]

@piotrkot they're still visible here

[piotrkot]

@llorllale Well, I don't know how to do it, then.

[0crat]

Job gh:yegor256/cactoos#1073 is not assigned, can't get performer

[0crat]

This job is not in scope