Skip to content

Conversation

Evalir
Copy link
Member

@Evalir Evalir commented Sep 10, 2025

Implements FromEnv manually for the calculator. It now has the following behavior:

  • First, the CHAIN_NAME environment variable is checked. If it's present, instantiation from the chain name is attempted.
  • If the CHAIN_NAME environment variable is not present, the individual environment variables used to configure the calculator are looked up, and the calculator is instantiated. Any of these missing results in a hard error.

Implements `FromEnv` manuall for the calculator. It now has the following behavior:

- First, the `CHAIN_NAME` environment variable is checked. If it's present, instantiation from the chain name is attempted.
- If the `CHAIN_NAME` environment variable is not present, the individual environment variables used to configure the calculator are looked up, and the calculator is instantiated. Any of these missing results in a hard error.
Copy link
Member Author

Evalir commented Sep 10, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Evalir Evalir requested a review from prestwich September 10, 2025 19:55
@Evalir Evalir requested a review from prestwich September 10, 2025 19:58
@Evalir Evalir force-pushed the evalir/manual-calc-fromenv branch from b6775cc to c6eafce Compare September 10, 2025 19:59
@Evalir Evalir force-pushed the evalir/manual-calc-fromenv branch from c6eafce to 7ccb988 Compare September 10, 2025 19:59
@Evalir Evalir enabled auto-merge (squash) September 10, 2025 20:01
@Evalir Evalir merged commit c0c5508 into main Sep 10, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants