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

Fix make exec-hash script #425

Open
SidestreamColdMelon opened this issue Sep 3, 2024 · 0 comments · May be fixed by #424
Open

Fix make exec-hash script #425

SidestreamColdMelon opened this issue Sep 3, 2024 · 0 comments · May be fixed by #424
Assignees

Comments

@SidestreamColdMelon
Copy link
Contributor

Goal

exec-hash script is working or related check from the reviewer checklist is removed

Context

  • Running script currently returns error on mac
date: illegal option -- d
usage: date [-jnRu] [-I[date|hours|minutes|seconds]] [-f input_fmt]
            [-r filename|seconds] [-v[+|-]val[y|m|w|d|H|M|S]]
            [[[[mm]dd]HH]MM[[cc]yy][.SS] | new_date] [+output_fmt]
  • Additional problem is inconsistent exec-doc name
    • Executive Vote in exec doc name will make the script fail (Error: Executive vote not found) as the script was develop to work with Executive vote

Potential solutions

  • Rewrite script from shell to another language (eg python)
  • Simplify script (eg do not try to find exec doc in the repo)
  • Remove script and relevant checklist
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 a pull request may close this issue.

2 participants