django-cms / djangocms-frontend

django CMS frontend is a plugin bundle for django CMS providing several components for the frontend, currently implemented with the popular Bootstrap 5 framework.
Other
43 stars 20 forks source link

fix: optional djangocms-icon dependency #36

Closed fsbraun closed 2 years ago

fsbraun commented 2 years ago

So far djangocms-frontend required djangocms-icon to be installed. This becomes a soft dependency: If not installed, djangocms-frontend will not offer icons as decorator for links. To install djangocms-icon with djangocms-frontend, the requirement djangocms-frontend[djangocms-icon] is introduced.

codecov[bot] commented 2 years ago

Codecov Report

Merging #36 (fc02295) into master (4e9101c) will decrease coverage by 0.02%. The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
- Coverage   81.65%   81.62%   -0.03%     
==========================================
  Files         128      128              
  Lines        3962     3968       +6     
  Branches      796      798       +2     
==========================================
+ Hits         3235     3239       +4     
- Misses        600      601       +1     
- Partials      127      128       +1     
Impacted Files Coverage Δ
djangocms_frontend/contrib/link/cms_plugins.py 80.95% <50.00%> (+0.46%) :arrow_up:
djangocms_frontend/contrib/link/forms.py 82.51% <66.66%> (-0.82%) :arrow_down:
djangocms_frontend/settings.py 96.42% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 4e9101c...fc02295. Read the comment docs.

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 175cc8a393c83e4dc3bc8d9dd488293c3fc9061d into af81fd64ab5b4ae6154f37d30fb597f2dfaf4980 - view on LGTM.com

new alerts:

fsbraun commented 2 years ago

@marksweb Does this conditional dependency on djangocms-icon look ok?

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging cc604dcf83c21cad812beee8e5b1bfe4ee14387d into edc3480b1abbfc3eb3ce1a103e85fca92c7481d1 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging cc3e1e270f192515240a1a9102c8817711cfdf3c into edc3480b1abbfc3eb3ce1a103e85fca92c7481d1 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 58ce3f30f07f9d14e8719b728eb829ecffaef75f into 4e9101c95adae13beee1814bae9f0430557b02cc - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging fc02295f4594f22abce8c0514068f6a427fac051 into 4e9101c95adae13beee1814bae9f0430557b02cc - view on LGTM.com

new alerts: