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

Refactor codec registry + remove consts package #1219

Closed
joshua-kim opened this issue Jul 30, 2024 · 1 comment · Fixed by #1258
Closed

Refactor codec registry + remove consts package #1219

joshua-kim opened this issue Jul 30, 2024 · 1 comment · Fixed by #1258

Comments

@joshua-kim
Copy link
Contributor

We have a strange structure in the packages where the codecs aren't even initialized in their own package. Need to move the initialization code to the same place the codecs are defined.

@aaronbuchwald
Copy link
Collaborator

Note: the intention here is to ensure that the action/auth registries can be imported from any package. We should still get rid of this because it's bizarre that the consts/ package would require importing a different package to make sure that it's correctly initialized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants