Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generate emails from eligible voters in voters.yaml #59

Merged
merged 2 commits into from
May 9, 2022

Conversation

geekygirldawn
Copy link
Contributor

Created a script that takes an Elekto voters.yaml file and a GitHub org name as input to generate a list of emails for registered voters. I'm contributing it to Elekto as suggested by @jberkus.

A few notes and things you might want me to change:

  • I created a scripts directory for this in what seemed like a logical place (to me), but let me know if you want me to move it.
  • Right now, I'm reading in the GitHub personal access token as keyboard input, which is less than ideal, but easy. I'm happy to change this if you let me know your preferred method. We should keep in mind that the person running the script is likely an election officer, and not necessarily the person running the Elekto instance, and they are probably running it locally, rather than on the server where Elekto is running, which probably impacts how we want to get the key.

…rg name as input to generate a list of emails for registered voters.
…This new version is ~3.5 times faster, and is able to find more email addresses. The original version found ~85% of the email addresses for the Knative 2022 TOC voters compared to this version, which found 96% of the email addresses.
@jberkus
Copy link
Member

jberkus commented May 9, 2022

I think this is good enough as an example script. If folks want to add refinements, they can do that as additional PRs.

@jberkus jberkus merged commit b2c6dd8 into elekto-io:main May 9, 2022
@geekygirldawn geekygirldawn deleted the emails branch May 23, 2022 14: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.

2 participants