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

Transition to uber fx #246

Closed
kristinapathak opened this issue Jan 22, 2022 · 2 comments
Closed

Transition to uber fx #246

kristinapathak opened this issue Jan 22, 2022 · 2 comments
Assignees
Projects

Comments

@kristinapathak
Copy link
Contributor

In general, we want to transition our applications over to the uber fx framework. Since tr1d1um is relatively small and straightforward, we want to start here. This is also a good opportunity to move away from the webpa-common library, which is slowly being deprecated. Below are some of the things we will need to transition:

  • Use arrange to help with configuration and other uber fx wiring.
  • Move to zap logging. Sallust will help with this.
  • Use basculehttp Provide functions for easier setup of bascule middleware.
  • Use touchstone for prometheus compatible metrics.
  • Set up http servers and clients with httpaux.
  • No longer import webpa-common directly into any tr1d1um package. After these other changes, the only package remaining should be device. We will need to either move the functionality to a new module (this is my preference) or put the functionality we need into tr1d1um directly.
@kristinapathak kristinapathak added this to New issues in XMiDT via automation Jan 22, 2022
@kristinapathak
Copy link
Contributor Author

Talked with @schmidtw and @johnabass and we tentatively agreed on moving device ID functionality to wrp-go.

@denopink denopink self-assigned this Sep 1, 2022
@denopink
Copy link
Contributor

denopink commented Sep 1, 2022

closed by #295

@denopink denopink closed this as completed Sep 1, 2022
XMiDT automation moved this from New issues to Done Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
XMiDT
  
Done
Development

No branches or pull requests

3 participants