Skip to content

refactor(ConfigChanges): use for-of loop for iterating over array#159

Merged
raphinesse merged 1 commit into
apache:masterfrom
raphinesse:cc-for-of
Mar 2, 2021
Merged

refactor(ConfigChanges): use for-of loop for iterating over array#159
raphinesse merged 1 commit into
apache:masterfrom
raphinesse:cc-for-of

Conversation

@raphinesse
Copy link
Copy Markdown
Contributor

Just a tiny code smell I stumbled upon

@raphinesse raphinesse added this to the 4.0.3 milestone Mar 1, 2021
@raphinesse raphinesse requested a review from breautek March 1, 2021 21:54
@codecov-io
Copy link
Copy Markdown

codecov-io commented Mar 1, 2021

Codecov Report

Merging #159 (8084a77) into master (7e615a5) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #159   +/-   ##
=======================================
  Coverage   88.24%   88.24%           
=======================================
  Files          20       20           
  Lines        1174     1174           
=======================================
  Hits         1036     1036           
  Misses        138      138           
Impacted Files Coverage Δ
src/ConfigChanges/ConfigChanges.js 96.89% <100.00%> (ø)

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 7e615a5...8084a77. Read the comment docs.

@raphinesse raphinesse merged commit 5ad1bdb into apache:master Mar 2, 2021
@raphinesse raphinesse deleted the cc-for-of branch March 2, 2021 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants