-
Notifications
You must be signed in to change notification settings - Fork 4
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
Logfmt: add support for new structure and new features (flags, arguments) #55
Conversation
Good point. I thought about this and I would lean into a major version if I was asked, due to the amount of breaking changes. BUT. We're the only users AFAIK, so I'm also not sure if it's worth overthinking it. |
Definitely should have been a major version increase. |
Ready to revert the revert. |
Even though our current CHANGELOG.md is not great, do you mind adding a new entry there? |
Updated, let me know what you think @svennergr |
That's great - thank you! |
Reverts #54
Adds this back. We shouldn't merge yet because we should figure out if this shouldn't be a major version increase?