Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add new pull request formatting check workflow #10196

Merged
merged 2 commits into from Nov 30, 2023

Conversation

nathanklick
Copy link
Member

Description

This pull request changes the following:

  • Adds a new CI workflow to enforce all pull request titles to follow the conventional commit syntax.

Related Issues

Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
@nathanklick nathanklick self-assigned this Nov 30, 2023
@nathanklick nathanklick requested review from a team as code owners November 30, 2023 18:35
@nathanklick nathanklick changed the title Pull Request Formatting Check chore: add new pull request formatting check workflow Nov 30, 2023
Copy link

Node: Unit Test Results

    2 288 files  ±0      2 288 suites  ±0   46m 11s ⏱️ + 3m 30s
118 405 tests ±0  118 371 ✔️ ±0  34 💤 ±0  0 ±0 
126 777 runs  ±0  126 743 ✔️ ±0  34 💤 ±0  0 ±0 

Results for commit c38bcd0. ± Comparison against base commit f2aac3a.

Copy link

codecov bot commented Nov 30, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (develop@26c6a1f). Click here to learn what that means.

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

Additional details and impacted files
@@            Coverage Diff             @@
##             develop   #10196   +/-   ##
==========================================
  Coverage           ?   63.16%           
  Complexity         ?    30712           
==========================================
  Files              ?     3328           
  Lines              ?   133955           
  Branches           ?    13884           
==========================================
  Hits               ?    84619           
  Misses             ?    45990           
  Partials           ?     3346           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

Node: HAPI Test (Crypto) Results

211 tests   201 ✔️  21m 2s ⏱️
  22 suites    10 💤
  22 files        0

Results for commit c38bcd0.

Copy link

Node: HAPI Test (Token) Results

190 tests   188 ✔️  21m 8s ⏱️
  13 suites      2 💤
  13 files        0

Results for commit c38bcd0.

Copy link

Node: HAPI Test (Misc) Results

419 tests   312 ✔️  24m 23s ⏱️
  73 suites  106 💤
  73 files        1

For more details on these failures, see this check.

Results for commit c38bcd0.

Copy link

Node: HAPI Test (Time Consuming) Results

21 tests     9 ✔️  25m 22s ⏱️
  2 suites  12 💤
  2 files      0

Results for commit c38bcd0.

Copy link

Node: E2E Test Results

    1 files      1 suites   22m 57s ⏱️
311 tests 311 ✔️ 0 💤 0
333 runs  333 ✔️ 0 💤 0

Results for commit c38bcd0.

Copy link

Node: Integration Test Results

279 tests   279 ✔️  28m 22s ⏱️
    5 suites      0 💤
    5 files        0

Results for commit c38bcd0.

Copy link

Node: HAPI Test (Smart Contract) Results

411 tests   321 ✔️  44m 1s ⏱️
  56 suites    90 💤
  56 files        0

Results for commit c38bcd0.

@nathanklick nathanklick merged commit fe032d4 into develop Nov 30, 2023
14 of 16 checks passed
@nathanklick nathanklick deleted the 10180-PR-Title-Check branch November 30, 2023 21:56
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.

Add a new PR Title enforcement workflow and make it a required PR check
2 participants