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

#862 Enforce immutability of IterableEnvelope through its iterator #890

Merged
merged 1 commit into from
May 29, 2018
Merged

#862 Enforce immutability of IterableEnvelope through its iterator #890

merged 1 commit into from
May 29, 2018

Conversation

Isammoc
Copy link
Contributor

@Isammoc Isammoc commented May 19, 2018

Pull request for #862

@0crat 0crat added the scope label May 19, 2018
@0crat
Copy link
Collaborator

0crat commented May 19, 2018

Job #890 is now in scope, role is REV

@codecov-io
Copy link

codecov-io commented May 19, 2018

Codecov Report

Merging #890 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #890      +/-   ##
============================================
+ Coverage     87.09%   87.09%   +<.01%     
  Complexity     1457     1457              
============================================
  Files           259      259              
  Lines          3773     3774       +1     
  Branches        212      212              
============================================
+ Hits           3286     3287       +1     
  Misses          439      439              
  Partials         48       48
Impacted Files Coverage Δ Complexity Δ
...in/java/org/cactoos/iterable/IterableEnvelope.java 100% <100%> (ø) 2 <1> (ø) ⬇️

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 a1e27f3...1d88caf. Read the comment docs.

@Isammoc Isammoc changed the title Enforce immutability of IterableEnvelope through its iterator #862 Enforce immutability of IterableEnvelope through its iterator May 19, 2018
/**
* Test case for {@link IterableEnvelope}.
*
* @since 0.34
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Isammoc we're on 0.35 now

@llorllale
Copy link
Contributor

@Isammoc just one minor comment, see above

@Isammoc
Copy link
Contributor Author

Isammoc commented May 29, 2018

@llorllale Done

@llorllale
Copy link
Contributor

@rultor merge

@rultor
Copy link
Collaborator

rultor commented May 29, 2018

@rultor merge

@llorllale OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 1d88caf into yegor256:master May 29, 2018
@rultor
Copy link
Collaborator

rultor commented May 29, 2018

@rultor merge

@llorllale Done! FYI, the full log is here (took me 9min)

@0crat 0crat removed the scope label May 30, 2018
@0crat
Copy link
Collaborator

0crat commented May 30, 2018

The job #890 is now out of scope

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants