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

Introduce multi files for authorization definitions #602

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

skelz0r
Copy link
Member

@skelz0r skelz0r commented Dec 12, 2024

No description provided.

@skelz0r skelz0r self-assigned this Dec 12, 2024
Base automatically changed from features/enhance-generator to develop December 12, 2024 13:49
@skelz0r skelz0r force-pushed the features/split-definition-file branch from 0a65309 to 0037e3c Compare December 12, 2024 13:58
@JeSuisUnCaillou
Copy link
Contributor

C'est approved, on merge ? :D

@skelz0r
Copy link
Member Author

skelz0r commented Dec 19, 2024

Vas-y ;p

@skelz0r
Copy link
Member Author

skelz0r commented Dec 19, 2024

(Plus sérieusement c'est un merge conflict relou, faut que je passe dessus mais y'a clairement milles trucs plus important ou urgent)

@JeSuisUnCaillou
Copy link
Contributor

Je pense que #622 devrait résoudre le conflit

@skelz0r
Copy link
Member Author

skelz0r commented Dec 19, 2024

Je pense que pour le coup non et que t'as fait un détour pour résoudre le problème, mais soit :D

@skelz0r
Copy link
Member Author

skelz0r commented Dec 19, 2024

(en fait t'auras toujours le conflit, mais il sera plus simple à résoudre)

@JeSuisUnCaillou
Copy link
Contributor

Oui, c'est vrai. Le résoudre reviendra à dire "ok on peut delete le fichier authorization_definitions.yml" quoi

@skelz0r
Copy link
Member Author

skelz0r commented Dec 19, 2024

Et en fait le merge d'avant était un simple cp

@JeSuisUnCaillou
Copy link
Contributor

Et en fait le merge d'avant était un simple cp

C'est ce que j'ai fait finalement, j'ai copié les définitions manquantes du fichier de définitions de develop dans le fichier de définitions/base de cette branche.

C'est pas ce qu'il fallait faire ?

@skelz0r
Copy link
Member Author

skelz0r commented Dec 19, 2024

Si, mais directement dans un rebase aurait été plus cohérent pour avoir des commits atomiques verts

@JeSuisUnCaillou
Copy link
Contributor

JeSuisUnCaillou commented Dec 19, 2024

J'avais fait la branche pour bien voir les diffs et comprendre ce que je devais faire, donc soit. Je vais juste rebase ici et faire la même chose que j'ai fait dans ma branche mais dans ton commit.

@JeSuisUnCaillou
Copy link
Contributor

J'ai pas travaillé dessus, et selon toute probabilité je travaillerai pas dessus avant un moment. Donc si quelqu'un veut reprendre, c'est autorisé ;)

@skelz0r
Copy link
Member Author

skelz0r commented Jan 6, 2025

Je cleanerai tkt

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.

3 participants