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.

Sign up for GitHub

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

Jump to bottom

ci: replace custom lint job with pre-commit/action #512

Merged
merged 3 commits into from
Jul 23, 2024
Merged

ci: replace custom lint job with pre-commit/action #512

merged 3 commits into from
Jul 23, 2024

Conversation

afuetterer
Copy link
Contributor

@afuetterer afuetterer commented Jul 10, 2024 •
edited
Loading

Description

This PR replaces the custom lint job with the official pre-commit/action, which takes care of caching as well.

Types of changes

  • Other (please describe): CI

Checklist

@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 10, 2024 •
edited
Loading

📋 Pytest Results

19 tests  ±0   19 ✅ ±0   23s ⏱️ -2s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 53095c9. ± Comparison against base commit 9f419b8.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

📋 Code Coverage

Code Coverage

Package Line Rate Branch Rate Health
. 58% 35% ➖
config 100% 100% ✔
controllers 85% 56% ✔
evaluators 64% 38% ➖
harvester 60% 51% ➖
helper 65% 55% ➖
models 80% 87% ✔
Summary 70% (6836 / 9789) 60% (2874 / 4790) ➖

@afuetterer
Copy link
Contributor Author

@huberrob: Preparing the switch to coveralls. Can you merge this PR?

@afuetterer afuetterer mentioned this pull request Jul 19, 2024
Closed
8 tasks
@huberrob huberrob merged commit 75c1199 into pangaea-data-publisher:master Jul 23, 2024
3 of 4 checks passed
@afuetterer afuetterer deleted the ci branch July 23, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@afuetterer @huberrob