agecem/babillard
4
0
Fork 0
Code Issues 3 Pull requests Packages Releases Activity

BREAKING: Déplacer config à cmd.app.Config et bump couleuvre #14

Merged
vlbeaudoin merged 3 commits from refactor/app-config into main 2023-12-04 21:32:39 -05:00
Conversation 0 Commits 3 Files changed 6 +20 -22
vlbeaudoin commented 2023-12-04 21:32:33 -05:00
Owner
Copy link
No description provided.
vlbeaudoin added 3 commits 2023-12-04 21:32:33 -05:00
a3c2c8d1f0 change!: remove config.Config dependency injection in ContenuHandler 
`ContenuHandler` now only stores a `ContenuDir string` instead of a
whole config.

BREAKING: change `handlers.RunServer` parameters to `contenuDir string, serverPort int`
72ff53f846 change!: move config to cmd.app.Config 
bump: couleuvre -> `v0.10.0`

Move `couleuvre.NewApp` to `cmd.init`

Make flags target `&cmd.app.Config`

Fix `ServerCmdExecuter`'s `RunServer` parameters

Fix call to `app.Parse` in `cmd.Execute`

BREAKING: Remove `cmd.Cfg()`
BREAKING: Remove `cmd.cfg config.Config`
vlbeaudoin merged commit 7c4e8d8227 into main 2023-12-04 21:32:39 -05:00
vlbeaudoin deleted branch refactor/app-config 2023-12-04 21:32:39 -05:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: agecem/babillard#14
No description provided.
Delete branch "refactor/app-config"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?