Skip to content

Create 36505#36516

Closed
pako81 wants to merge 1 commit into
masterfrom
pako81-patch-1-1
Closed

Create 36505#36516
pako81 wants to merge 1 commit into
masterfrom
pako81-patch-1-1

Conversation

@pako81
Copy link
Copy Markdown

@pako81 pako81 commented Dec 3, 2019

Created changelog item for #36505

@pako81 pako81 requested a review from micbar December 3, 2019 14:31
@pako81 pako81 self-assigned this Dec 3, 2019
Copy link
Copy Markdown
Contributor

@micbar micbar left a comment

Choose a reason for hiding this comment

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

Thx.

  • Actually this should go to the original PR
  • The first line should not exceed 80 chars
  • The first should be written in present tense
  • The summary should be written in past tense, like you already did.

@pako81
Copy link
Copy Markdown
Author

pako81 commented Dec 3, 2019

@micbar ops, sorry for the mistake. I have now done this in the original PR and also made the suggested changes. Closing this?

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 3, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #36516   +/-   ##
=========================================
  Coverage     64.68%   64.68%           
  Complexity    19024    19024           
=========================================
  Files          1268     1268           
  Lines         74369    74369           
  Branches       1311     1311           
=========================================
  Hits          48106    48106           
  Misses        25877    25877           
  Partials        386      386
Flag Coverage Δ Complexity Δ
#javascript 54.02% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 65.86% <ø> (ø) 19024 <ø> (ø) ⬇️

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 24443e5...6224d54. Read the comment docs.

@karakayasemi
Copy link
Copy Markdown
Contributor

closing favor of #36505.

@phil-davis phil-davis deleted the pako81-patch-1-1 branch December 4, 2019 02:26
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.

3 participants