Skip to content

Update Actions and Enable Dependabot for GH Actions#125

Open
wfondrie wants to merge 6 commits into
mainfrom
feature/dependabot
Open

Update Actions and Enable Dependabot for GH Actions#125
wfondrie wants to merge 6 commits into
mainfrom
feature/dependabot

Conversation

@wfondrie

@wfondrie wfondrie commented Sep 6, 2024

Copy link
Copy Markdown
Owner

This PR fixes the current actions and adds support for dependabot.

This also fixes the linting issues in main.

@wfondrie
wfondrie requested a review from jspaezp September 6, 2024 22:25
@codecov

codecov Bot commented Sep 6, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 11.11111% with 8 lines in your changes missing coverage. Please review.

Project coverage is 84.57%. Comparing base (6b8fbfd) to head (eb12c3f).

Files with missing lines Patch % Lines
mokapot/parsers/pin_to_tsv.py 0.00% 6 Missing ⚠️
mokapot/mokapot.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
- Coverage   84.60%   84.57%   -0.04%     
==========================================
  Files          26       26              
  Lines        2949     2943       -6     
==========================================
- Hits         2495     2489       -6     
  Misses        454      454              

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

@jspaezp

jspaezp commented Sep 6, 2024

Copy link
Copy Markdown
Collaborator

I think T20 can be disabled in everything under the docs dir ...

@jspaezp jspaezp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, we can fix it later on the files that are making it not pass.
FYI: #126 adds the new actions (except for dependabot)

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.

2 participants