Skip to content

Add ctfconfig.yml to ctf init.#107

Merged
Res260 merged 3 commits into
mainfrom
ctfconfig
Jun 1, 2026
Merged

Add ctfconfig.yml to ctf init.#107
Res260 merged 3 commits into
mainfrom
ctfconfig

Conversation

@Res260
Copy link
Copy Markdown
Collaborator

@Res260 Res260 commented May 31, 2026

Add ctfconfig.yml to ctf init.
This config file allows disabling validators and setting the CTF name. Implements #42

Signed-off-by: Émilio Gonzalez little.moon6016@fastmail.com

Comment thread ctf/commands/init.py
Comment thread ctf/common/validators.py Outdated
Comment thread ctf/common/validators.py
@MOBergeron
Copy link
Copy Markdown
Collaborator

Si le ctfconfig est obligatoire, on pourrait le charger comme un pydantic settings puis on pourrait mettre quelques config dedans comme remote, project, vm_remote, vm_project, etc.. Puis on pourrait avoir un override de dev, production, RingZer0, etc. C'est une suggestion, j'en ai fait plusieurs dans des projets, je peux m'en charger une fois la PR fermée si tu veux.

@Res260
Copy link
Copy Markdown
Collaborator Author

Res260 commented May 31, 2026

Si le ctfconfig est obligatoire, on pourrait le charger comme un pydantic settings puis on pourrait mettre quelques config dedans comme remote, project, vm_remote, vm_project, etc.. Puis on pourrait avoir un override de dev, production, RingZer0, etc. C'est une suggestion, j'en ai fait plusieurs dans des projets, je peux m'en charger une fois la PR fermée si tu veux.

Ouais c'est des bonnes idées, mais en effet après la PR initiale :p

…validators and setting the CTF name. Implements #42

Signed-off-by: Émilio Gonzalez <little.moon6016@fastmail.com>
Comment thread ctf/common/models.py Outdated
Comment thread ctf/common/models.py Outdated
Comment thread ctf/common/models.py
Res260 added 2 commits May 31, 2026 20:12
Signed-off-by: Émilio Gonzalez <little.moon6016@fastmail.com>
Signed-off-by: Émilio Gonzalez <little.moon6016@fastmail.com>
@Res260
Copy link
Copy Markdown
Collaborator Author

Res260 commented Jun 1, 2026

merging, si t'as d'autres commentaires j'adresserai plus tard

@Res260 Res260 merged commit 7aa388d into main Jun 1, 2026
7 of 9 checks passed
@Res260 Res260 deleted the ctfconfig branch June 1, 2026 22:56
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.

2 participants