Setting Electricity mix and providers from config file#108
Conversation
|
@NP4567-dev I think you have to run a |
|
I think there's a bigger problem than a Let me check this! |
|
@NP4567-dev It's fixed you can merge with the |
Nice,thank you for the fix! |
adrienbanse
left a comment
adrienbanse
left a comment
There was a problem hiding this comment.
@NP4567-dev Many thanks for this great contrib :)
Just next time I think it's safer to wait for a review to be done before merging (sorry if it wasn't clear, but I just added the CI comment without looking at the rest of the PR)
| print(f"GHG emissions: {response.impacts.gwp.value} kgCO2eq") | ||
| ``` | ||
|
|
||
| You can also provide the ecologits configuration through a toml file. |
There was a problem hiding this comment.
Here we could add that it has to be in a [ecologits] section
There was a problem hiding this comment.
So sorry, I misinterpreted the "you can merge with main" 😅
I don't have access but in the settings -> branch rules you can block merges to main until there is at least one approval (owners can still bypass)
There was a problem hiding this comment.
Oh right my bad it had double sense :/
I'll change this in the settings thanks!
This reverts commit de34f2e.
This reverts commit de34f2e.
2 participants
First part of using a file config as detailed in #98
For now it is just the basic case of setting provider and electricity mix zone.
Let me know if you want anything changed.