Skip to content

Conversation

@ygnice
Copy link
Contributor

@ygnice ygnice commented Mar 28, 2022

Your checklist for this pull request

🚨Please review the guidelines for contributing to this repository.

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Docs update
  • Code style update
  • Refactor
  • Performance Improvements
  • Build-related changes
  • Other, please describe:

Does this PR introduce a BREAKING CHANGE? (check one)

  • Yes
  • No

The PR fulfills these requirements:

  • Make sure you are requesting to pull a feature/bugfix branch (right side). Don't request master!
  • Make sure you are making a pull request against the develop branch (left side). Also you should start your branch off our develop branch.
  • Check the commit's or even all commits' message styles matches our requested structure.
  • Check your code additions will fail neither code linting checks nor unit test.
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Description

Please describe your pull request.

❤️ Thank you!

@ygnice ygnice requested a review from a team as a code owner March 28, 2022 03:50
@ygnice ygnice requested a review from Teakowa March 28, 2022 03:50
@codecov
Copy link

codecov bot commented Mar 28, 2022

Codecov Report

Merging #8 (7d9afe7) into main (ead6776) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main       #8   +/-   ##
=========================================
  Coverage     61.01%   61.01%           
  Complexity       18       18           
=========================================
  Files             4        4           
  Lines            59       59           
=========================================
  Hits             36       36           
  Misses           23       23           

@Teakowa Teakowa merged commit a88de0c into main Mar 28, 2022
@Teakowa Teakowa deleted the chore/workflows-tests branch March 30, 2022 03:50
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