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

Replace shell scripts with JS code #932

Open
lidel opened this issue Oct 20, 2020 · 3 comments
Open

Replace shell scripts with JS code #932

lidel opened this issue Oct 20, 2020 · 3 comments
Assignees
Labels
effort/hours Estimated to take one or several hours exp/intermediate Prior experience is likely helpful kind/maintenance Work required to avoid breaking changes or harm to project's status quo P2 Medium: Good to have, but can wait until someone steps up status/ready Ready to be worked

Comments

@lidel
Copy link
Member

lidel commented Oct 20, 2020

@lidel lidel added kind/maintenance Work required to avoid breaking changes or harm to project's status quo exp/intermediate Prior experience is likely helpful P2 Medium: Good to have, but can wait until someone steps up status/ready Ready to be worked effort/hours Estimated to take one or several hours labels Oct 20, 2020
@lidel lidel self-assigned this Oct 20, 2020
@BrianBelhumeur
Copy link

I can take this one. The json package seems like a good candidate.

To be clear, the goal is to replace the just the jq util, rather than the entire ci/update-manifest.sh shell script, correct?

@lidel
Copy link
Member Author

lidel commented Oct 26, 2020

@BrianBelhumeur Ideally we would replace entire update-manifest.sh with update-manifest.js that does the same but removes the need for shell scripts, but if we remove jq that also good enough, gets us 80% where we want to be :)

@BrianBelhumeur
Copy link

Hey @lidel I'm actually going to hand this one back. I don't currently have the bandwidth for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort/hours Estimated to take one or several hours exp/intermediate Prior experience is likely helpful kind/maintenance Work required to avoid breaking changes or harm to project's status quo P2 Medium: Good to have, but can wait until someone steps up status/ready Ready to be worked
Projects
No open projects
Status: Needs Grooming
Development

No branches or pull requests

2 participants