Skip to content

fix: module loading in retrieve_vote_script#469

Open
zkpepe wants to merge 1 commit intolidofinance:masterfrom
zkpepe:patch-1
Open

fix: module loading in retrieve_vote_script#469
zkpepe wants to merge 1 commit intolidofinance:masterfrom
zkpepe:patch-1

Conversation

@zkpepe
Copy link

@zkpepe zkpepe commented Aug 28, 2025

removed the use of exec and locals() inside retrieve_vote_script.
this was unreliable, since exec doesn’t guarantee variables end up in locals() when called from within a function.

now the code uses importlib.import_module instead, which cleanly imports the module and provides direct access to start_vote and get_vote_items.
this makes the logic safer and more predictable.

@zkpepe zkpepe requested a review from a team as a code owner August 28, 2025 17:00
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.

1 participant