Skip to content

Conversation

@ylsGit
Copy link
Contributor

@ylsGit ylsGit commented Apr 24, 2023

  • Targeted PR against correct branch (see CONTRIBUTING.md)

  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

  • Wrote tests

  • Updated relevant documentation (docs/)

  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md

  • Reviewed Files changed in the github PR explorer


For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge PR #XYZ: [title]" (coding standards)

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (dev@12e5bde). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 913c5d2 differs from pull request most recent head fb7c2ac. Consider uploading reports for the commit fb7c2ac to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@          Coverage Diff           @@
##             dev     #102   +/-   ##
======================================
  Coverage       ?   30.82%           
======================================
  Files          ?       52           
  Lines          ?     5453           
  Branches       ?        0           
======================================
  Hits           ?     1681           
  Misses         ?     3604           
  Partials       ?      168           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@xiangjianmeng xiangjianmeng merged commit 12168ea into dev Apr 24, 2023
giskook pushed a commit that referenced this pull request May 23, 2023
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.

5 participants