Skip to content

docs: add pr template#306

Merged
cristianmtr merged 1 commit intomainfrom
docs-pr-template
Apr 27, 2022
Merged

docs: add pr template#306
cristianmtr merged 1 commit intomainfrom
docs-pr-template

Conversation

@cristianmtr
Copy link
Copy Markdown
Contributor

  • add a PR template
  • allows for mentioning the goals of a PR
  • reminder for checking and updating docs, if required

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 27, 2022

Codecov Report

Merging #306 (454e2f2) into main (363a2fd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #306   +/-   ##
=======================================
  Coverage   86.29%   86.29%           
=======================================
  Files         134      134           
  Lines        6312     6312           
=======================================
  Hits         5447     5447           
  Misses        865      865           
Flag Coverage Δ
docarray 86.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 363a2fd...454e2f2. Read the comment docs.

Copy link
Copy Markdown
Contributor

@davidbp davidbp left a comment

Choose a reason for hiding this comment

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

LGTM

@cristianmtr cristianmtr merged commit 6e52701 into main Apr 27, 2022
@cristianmtr cristianmtr deleted the docs-pr-template branch April 27, 2022 14:31
davidbp pushed a commit that referenced this pull request Apr 28, 2022
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.

2 participants