django-cms / djangocms-transfer

django CMS Transfer allows you to export and import plugins.
https://www.django-cms.org
Other
19 stars 19 forks source link

pre-commit hook updates #25

Open bencipher opened 1 year ago

bencipher commented 1 year ago

Description

Related resources

Checklist

codecov[bot] commented 1 year ago

Codecov Report

Merging #25 (8c85f39) into master (b11589f) will decrease coverage by 0.10%. The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
- Coverage   32.19%   32.10%   -0.10%     
==========================================
  Files          11       11              
  Lines         382      380       -2     
  Branches       56       56              
==========================================
- Hits          123      122       -1     
+ Misses        259      258       -1     
Impacted Files Coverage Δ
djangocms_transfer/cms_plugins.py 35.36% <50.00%> (-0.35%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

bencipher commented 1 year ago

Okay, doing it now.

On Tue, 1 Nov 2022 at 21:03, Mark Walker @.***> wrote:

@.**** requested changes on this pull request.

In .pre-commit-config.yaml https://github.com/django-cms/djangocms-transfer/pull/25#discussion_r1010847131 :

@@ -0,0 +1,44 @@ +repos:

Could you add the ci config from here please?

https://github.com/django-cms/django-cms/blob/develop/.pre-commit-config.yaml

And you can just remove the black section rather than comment out as we've no plans to use black

— Reply to this email directly, view it on GitHub https://github.com/django-cms/djangocms-transfer/pull/25#pullrequestreview-1163967106, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACIB3UKGLRC7E66N2VOUOPLWGFZR3ANCNFSM6AAAAAARSUJJL4 . You are receiving this because you authored the thread.Message ID: @.***>