Skip to content

Conversation

@vcastellm
Copy link
Contributor

@vcastellm vcastellm commented Oct 29, 2025

Description

This is a follow-up on my last PR, it will try to load the env file from the specified path or fallback to search in the root project.

This also allows the fetch-l2oo-config to work standalone.

Note

This change is backwards compatible

Copy link
Collaborator

@Farhad-Shabani Farhad-Shabani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please apply the same change to the config and fetch-fault-dispute-game-config bins as well?

@Farhad-Shabani
Copy link
Collaborator

@vcastellm could you also make the first load (default one) from the root? if not found then try dotenv::from_path(args.env_file.clone())?

If project root isn't found, still attempt to load the provided env file
before emitting a warning to avoid spurious messages when the file
exists.
@vcastellm
Copy link
Contributor Author

Thanks for the feedback @Farhad-Shabani, I think it's better now

@Farhad-Shabani
Copy link
Collaborator

Of course @vcastellm
Closing this and reopening as #667 so all CI jobs can run.
Also it might stay a bit. We are seeing an issue on head, will be fixing that first, then merge this one.

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