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

[chore] Remplace black par ruff-formatter #1087

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

etchegom
Copy link
Collaborator

Il s'agit de :

  • Une correction de bug
  • Une nouvelle fonctionnalité programmée
  • Une nouvelle fonctionnalité spontanée

Décrivez vos changements

Remplacement de black par ruff-format

Checklist d'acceptation de revue de code

  • Aucun texte ne fait référence à du vocabulaire spécifique d'un métier
  • Mon code est auto-documenté ou la documentation a été mise à jour/créée
  • La gestion du multi-portail a été prise en compte
  • L'accessibilité a été prise en compte
  • Pre-commit est configuré et a été lancé
  • Des tests couvrant le code changé ont été ajoutés/modifiés
  • L'ensemble des tests front et back sont au vert

Demandes

  • Je souhaite un déploiement en préproduction

@etchegom etchegom force-pushed the chore/get-rid-of-black branch from 53bd541 to 9cc9dab Compare February 25, 2025 16:04
@etchegom etchegom changed the title [chore] Get rid of black [chore] Remplace black par ruff-formatter Feb 25, 2025
@@ -13,7 +13,7 @@ class OrganizationGroupAdmin(admin.ModelAdmin):

@admin.register(models.Organization)
class OrganizationAdmin(admin.ModelAdmin):
search_fields = ["name" "group__name"]
search_fields = ["name", "group__name"]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixé dans #1086

@@ -30,7 +30,7 @@ def ds_schema_sample():
"description": "Ce formulaire est destiné aux collectivités de Moselle qui souhaitent déposer une demande de subvention au titre de la DETR ou de la DSIL en 2024. Votre demande sera automatiquement orientée vers la sous-préfecture compétente.\r\n\r\nLa date limite de dépôt des dossiers est fixée au 30 novembre 2023.\r\n\r\nDans le cadre de l'instruction des demandes, les projets sont orientés par les sous-préfets en lien avec la préfecture vers la dotation la plus adaptée (DETR ou DSIL).\r\n\r\nPour l'éligibilité des opérations, il convient de se référer à l’appel à projets 2024 commun à la DETR et à la DSIL, disponible à l'adresse suivante : https://www.moselle.gouv.fr/Actions-de-l-Etat/Collectivites-controle-de-legalite-et-finances-locales/Subventions\r\n\r\n",
"required": False,
"collapsibleExplanationEnabled": True,
"collapsibleExplanationText": "\r\nPour constituer votre dossier, les sections suivantes sont à compléter :\r\n\r\n-Informations sur le porteur de projet\r\n-Présentation de l'opération\r\n-Plan de financement prévisionnel\r\n-Finalisation du dossier\r\n\r\n\r\n\u003Cbr /\u003E\r\n\r\nVotre dossier est enregistré automatiquement après chaque modification. Vous pouvez à tout moment fermer la fenêtre et reprendre plus tard là où vous en étiez.",
"collapsibleExplanationText": "\r\nPour constituer votre dossier, les sections suivantes sont à compléter :\r\n\r\n-Informations sur le porteur de projet\r\n-Présentation de l'opération\r\n-Plan de financement prévisionnel\r\n-Finalisation du dossier\r\n\r\n\r\n\u003cbr /\u003e\r\n\r\nVotre dossier est enregistré automatiquement après chaque modification. Vous pouvez à tout moment fermer la fenêtre et reprendre plus tard là où vous en étiez.",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Chelou ça, pas compris, mais ça passe.

@etchegom etchegom requested a review from glibersat February 25, 2025 16:10
@etchegom etchegom marked this pull request as ready for review February 25, 2025 16:10


[tool.ruff]
line-length = 88
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@etchegom etchegom self-assigned this Feb 25, 2025
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.

1 participant