Ad-hoc QOL updates #30
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
ZaradarBH
commented
Mar 16, 2023
ZaradarBH
commented
- Added missing "better-npm-run": "^0.1.1" package
- Clean up configuration files
- Clean up ENV variables
- Clean up docker tooling
- Split "collector" and "api" into seperate containers using a shared base image
- Add postgres service to docker-compose for convenience during local development
- Create a custom network for docker-compose to ensure we have automatic DNS resolution enabled (https://docs.docker.com/network/bridge/#differences-between-user-defined-bridges-and-the-default-bridge)
…ntainer images so they are easier to orchestrate individually
- Re-target to "@terra-money/terra.proto": "^0.1.7"
…zation Clean up container tooling to split API & collector
…zation moved docker-compose assets to seperate folder
…zation added docker-compose readme.md
merging upstream into our main
javiersuweijie
approved these changes
Mar 17, 2023
emidev98
approved these changes
Mar 20, 2023
- Add terra.proto.legacy to resolve upstream merge conflict - Add logic to pick the right PB decoder package based cfg.CHAIN_ID - Fix bug in docker-compose build context target - Add RPC endpoint that works
…ges-to-coexist Fix @terra-money/terra.proto conflict
…ackages-to-coexist
…ges-to-coexist Allow diff terre proto packages to coexist
|
I updated the PR with the latest changes and a suggestion for how to use diff versions of terra.proto at the same time. That should provide a way for us to workaround the dependency conflicts we have on the PB decoders. FYI the docker-compose file you added already exists here: https://github.com/classic-terra/fcd/blob/0b0383edc3c3f0551458bb3aae4fc481039f2a4f/docker-compose/docker-compose.yml#L4 . We can move the compose files to the root of the repo and merge whichever version of the postgres service registration you prefere. |
emidev98
reviewed
Mar 23, 2023
…ges-to-coexist Bump minor version due to ENV changes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.