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

Remove pre-commit GH action #997

Merged
merged 2 commits into from
Sep 19, 2023

Conversation

ogajduse
Copy link
Member

@ogajduse ogajduse commented Sep 19, 2023

This PR removes pre-commit/action in favor of pre-commit.ci that was enabled on the nailgun repository by @JacobCallahan.

@ogajduse ogajduse force-pushed the test/pre-commit-ci-intro branch from 749fb9c to 3960c1b Compare September 19, 2023 15:18
@ogajduse ogajduse force-pushed the test/pre-commit-ci-intro branch from a4ed081 to f51306d Compare September 19, 2023 15:33
@ogajduse
Copy link
Member Author

pre-commit.ci autofix

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01% 🎉

Comparison is base (4e4026f) 92.14% compared to head (43ebe49) 92.15%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #997      +/-   ##
==========================================
+ Coverage   92.14%   92.15%   +0.01%     
==========================================
  Files           6        6              
  Lines        3043     3047       +4     
==========================================
+ Hits         2804     2808       +4     
  Misses        239      239              
Files Changed Coverage Δ
nailgun/entities.py 91.30% <100.00%> (+0.01%) ⬆️

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

@ogajduse ogajduse force-pushed the test/pre-commit-ci-intro branch from aa64316 to 43ebe49 Compare September 19, 2023 15:57
@ogajduse ogajduse added 6.12.z 6.13.z 6.14.z CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing labels Sep 19, 2023
@ogajduse ogajduse changed the title pre-commit.ci testing Remove pre-commit GH action Sep 19, 2023
@ogajduse ogajduse marked this pull request as ready for review September 19, 2023 16:02
@JacobCallahan JacobCallahan merged commit 08c4bb8 into SatelliteQE:master Sep 19, 2023
19 of 23 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 19, 2023
* pre-commit.ci: Disable autofixing PRs

* Remove pre-commit action from GHA PR workflow

(cherry picked from commit 08c4bb8)
github-actions bot pushed a commit that referenced this pull request Sep 19, 2023
* pre-commit.ci: Disable autofixing PRs

* Remove pre-commit action from GHA PR workflow

(cherry picked from commit 08c4bb8)
github-actions bot pushed a commit that referenced this pull request Sep 19, 2023
* pre-commit.ci: Disable autofixing PRs

* Remove pre-commit action from GHA PR workflow

(cherry picked from commit 08c4bb8)
ColeHiggins2 pushed a commit to ColeHiggins2/nailgun that referenced this pull request Oct 3, 2023
* pre-commit.ci: Disable autofixing PRs

* Remove pre-commit action from GHA PR workflow
@ogajduse ogajduse deleted the test/pre-commit-ci-intro branch October 11, 2023 15:33
JacobCallahan pushed a commit that referenced this pull request Oct 12, 2023
* pre-commit.ci: Disable autofixing PRs

* Remove pre-commit action from GHA PR workflow

(cherry picked from commit 08c4bb8)
JacobCallahan pushed a commit that referenced this pull request Oct 12, 2023
* pre-commit.ci: Disable autofixing PRs

* Remove pre-commit action from GHA PR workflow

(cherry picked from commit 08c4bb8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.12.z 6.13.z 6.14.z AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants