config.go should not depend on any libraries #107

Open
opened 2023-05-19 01:39:09 +00:00 by bvisness · 0 comments
Owner

Our config.go requires you to import a couple libraries in order to provide log levels and such. This has proven to be a problem, because changing dependency versions sometimes causes config files to break. (Your individual config.go file is excluded from git, after all.) I think we should remove the need for third-party libraries from config.go so that we can avoid this friction in the future.

I realize that in some cases this comes at a small complexity or UX cost (nobody likes looking up docs to see what magic strings mean what), but I think we can mitigate it by just shipping our own enums in the config package.

Our config.go requires you to import a couple libraries in order to provide log levels and such. This has proven to be a problem, because changing dependency versions sometimes causes config files to break. (Your individual config.go file is excluded from git, after all.) I think we should remove the need for third-party libraries from config.go so that we can avoid this friction in the future. I realize that in some cases this comes at a small complexity or UX cost (nobody likes looking up docs to see what magic strings mean what), but I think we can mitigate it by just shipping our own enums in the config package.
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
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: hmn/hmn#107
No description provided.