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 crispy-forms-foundation and django-crispy-forms from requirements #3211

Merged
merged 5 commits into from
Nov 22, 2024

Conversation

johannaengland
Copy link
Contributor

@johannaengland johannaengland commented Nov 15, 2024

Removes crispy-forms-foundation and django-crispy-forms from requirements and necessary settings for those. Also changes some wording to not directly reference crispy

@johannaengland johannaengland added cleanup dependencies Pull requests that update a dependency file labels Nov 15, 2024
Copy link

github-actions bot commented Nov 15, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON black 991 0 12.84s
✅ PYTHON ruff 987 0 0.1s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

Copy link

github-actions bot commented Nov 15, 2024

Test results

    9 files      9 suites   8m 16s ⏱️
2 139 tests 2 139 ✅ 0 💤 0 ❌
4 017 runs  4 017 ✅ 0 💤 0 ❌

Results for commit 05a6651.

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Nov 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.39%. Comparing base (6f508a6) to head (05a6651).
Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3211      +/-   ##
==========================================
- Coverage   60.39%   60.39%   -0.01%     
==========================================
  Files         605      605              
  Lines       43685    43683       -2     
  Branches       48       48              
==========================================
- Hits        26383    26381       -2     
  Misses      17290    17290              
  Partials       12       12              

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


🚨 Try these New Features:

@johannaengland johannaengland self-assigned this Nov 15, 2024
@johannaengland johannaengland marked this pull request as ready for review November 15, 2024 13:45
Base automatically changed from cleanup/rename-non-crispy-checkbox to cleanup/remove-unused-crispy-checkbox November 19, 2024 09:13
Base automatically changed from cleanup/remove-unused-crispy-checkbox to master November 19, 2024 09:15
@johannaengland johannaengland force-pushed the cleanup/finally-remove-crispy-forms branch from 0a988bd to 3034cd5 Compare November 19, 2024 09:17
@stveit
Copy link
Contributor

stveit commented Nov 19, 2024

Place templates for globally overriding crispy_forms_foundation templates in this directory.
reference here to crispy_forms_foundation

@johannaengland
Copy link
Contributor Author

johannaengland commented Nov 19, 2024

Place templates for globally overriding crispy_forms_foundation templates in this directory.

reference here to crispy_forms_foundation

True, generally the folder foundation-5 should get a new name or be merged with custom_crispy_templates.

I'll create a new issue for that since I think @podliashanyk better knows how to order the templates

#3216

@stveit
Copy link
Contributor

stveit commented Nov 19, 2024

It doesnt seem like this breaks anything, but more people should test

@lunkwill42
Copy link
Member

Closes #2794.

Is that correct? There is still at least one unchecked checkbox on #2794, and this PR does not alone close all of #2794...

@johannaengland johannaengland changed the title Completely remove crispy-forms Remove crispy-forms-foundation and django-crispy-forms from requirements Nov 20, 2024
Copy link
Member

@lunkwill42 lunkwill42 left a comment

Choose a reason for hiding this comment

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

The changes by themselves seem fine to me. All tests are green, and there doesn't seem to be any significant references to crispy-forms left in the codebase.

I'd like to see an entry about removed dependencies in NOTES.rst, though :)

@lunkwill42 lunkwill42 mentioned this pull request Nov 20, 2024
59 tasks
@lunkwill42 lunkwill42 force-pushed the cleanup/finally-remove-crispy-forms branch from 38b576e to 4b26832 Compare November 20, 2024 14:11
@lunkwill42
Copy link
Member

Sorry, accidentally pushed two more commits to the wrong branch - reset and force-pushed back to the last commit before my mistake.

@johannaengland
Copy link
Contributor Author

I'd like to see an entry about removed dependencies in NOTES.rst, though :)

I thought NOTES.rst will be updated when creating a release since we switched to towncrier?

@lunkwill42
Copy link
Member

I'd like to see an entry about removed dependencies in NOTES.rst, though :)

I thought NOTES.rst will be updated when creating a release since we switched to towncrier?

NOTES.rst are our project release notes. We use it to communicate changes and necessary actions to take during upgrades. towncrier has no bearing on that file: It is always authored by humans on a case-by-case basis. Have you read it? :-) I usually edit it at release-time, but it is much easier to forget to add stuff to it if it isn't done continuously.

We usually make a subsection of dependency changes for each release.

@johannaengland johannaengland force-pushed the cleanup/finally-remove-crispy-forms branch from ba55606 to f429b1a Compare November 22, 2024 12:43
@johannaengland johannaengland force-pushed the cleanup/finally-remove-crispy-forms branch from f429b1a to 05a6651 Compare November 22, 2024 12:57
Copy link

sonarcloud bot commented Nov 22, 2024

@johannaengland johannaengland merged commit 74edff4 into master Nov 22, 2024
12 checks passed
@johannaengland johannaengland deleted the cleanup/finally-remove-crispy-forms branch November 22, 2024 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants