Skip to content

Conversation

@TomerFi
Copy link
Owner

@TomerFi TomerFi commented Dec 26, 2024

Description

  • I refactored the CI process and am making minor fixes.
  • Small fixup to the community documents.
  • Added dependabot.
  • Removed unused bots (environments will be clean after the merge).
  • Small fixups to README.
  • Bumped all dev dependencies to the latest versions.

fixes: #797

Checklist

  • I have followed this repository's contributing guidelines.
  • I will adhere to the project's code of conduct.

Additional information

@pull-request-size pull-request-size bot added the size: xl Pull request has 500 to 1000 lines label Dec 26, 2024
@auto-me-bot auto-me-bot bot added the status: needs review Pull request needs a review label Dec 26, 2024
@TomerFi TomerFi force-pushed the better-ci branch 4 times, most recently from bc8b5ed to 1043b34 Compare December 26, 2024 05:42
@pull-request-size pull-request-size bot added size: l Pull request has 100 to 500 lines and removed size: xl Pull request has 500 to 1000 lines labels Dec 26, 2024
@TomerFi TomerFi force-pushed the better-ci branch 2 times, most recently from 34dea90 to a590894 Compare December 26, 2024 05:49
@github-actions
Copy link

github-actions bot commented Dec 26, 2024

Test Results

294 tests   294 ✅  2s ⏱️
  1 suites    0 💤
  1 files      0 ❌

Results for commit 7463523.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Dec 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.99%. Comparing base (4fbd944) to head (7463523).
Report is 3 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #814   +/-   ##
=======================================
  Coverage   98.99%   98.99%           
=======================================
  Files          11       11           
  Lines        1293     1293           
=======================================
  Hits         1280     1280           
  Misses         13       13           

@TomerFi TomerFi force-pushed the better-ci branch 2 times, most recently from 4157eb8 to d8f236b Compare December 26, 2024 14:41
Signed-off-by: Tomer Figenblat <[email protected]>
Copy link
Collaborator

@thecode thecode left a comment

Choose a reason for hiding this comment

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

Thanks,
Btw - feel free to merge CI PRs without review, however if needed I can look at them

@auto-me-bot auto-me-bot bot added status: approved Pull request is approved and removed status: needs review Pull request needs a review labels Dec 26, 2024
@TomerFi TomerFi changed the title ci: ci maintainance ci: workflow maintenance, bots cleanup, add dependabot Dec 26, 2024
@TomerFi TomerFi merged commit e29f579 into dev Dec 26, 2024
11 checks passed
@TomerFi TomerFi deleted the better-ci branch December 26, 2024 18:40
@auto-me-bot auto-me-bot bot added status: merged Pull request merged and removed status: approved Pull request is approved labels Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: l Pull request has 100 to 500 lines status: merged Pull request merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI workflows maintenance

3 participants