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

ci: sync pre-commit for v3 #7799

Open
wants to merge 21 commits into
base: release/3.11.x
Choose a base branch
from
Open

Conversation

marksweb
Copy link
Member

This synchronises pre-commit hooks from v4

@vinitkumar
Copy link
Member

@marksweb Is this valid anymore and should be do it for develop-4 instead or both?

sakhawy and others added 3 commits March 16, 2024 15:29
* feat: add RTL support to pagetree

- Use logical properties and `[dir=rtl]`
- Fix unsupported RTL styling in jsTree dnd marker
- Fix misc `changelist` RTL styling

* fix: remove double semi-colon

* fix: remove keyword values in background-position
* feat: new tests to validate response type depending on CMS_TOOLBAR_HIDE

* feat: return correct type from __call__ as defined by django
@@ -1 +1,2 @@
import nonexistingmodule # nopyflakes
# intentional bad import
import nonexistingmodule # noqa

Check notice

Code scanning / CodeQL

Unused import Note

Import of 'nonexistingmodule' is not used.
wfehr and others added 3 commits May 6, 2024 06:48
fix: also use key-length of 200 for the actual cache-key of placeholders
@fsbraun
Copy link
Sponsor Member

fsbraun commented May 20, 2024

@marksweb Can you reopen this PR against release/3.11.x?

@marksweb marksweb changed the base branch from develop to release/3.11.x May 20, 2024 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants