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 shellscirpt with ...? #825

Closed
kachick opened this issue Oct 8, 2024 · 5 comments · Fixed by #876
Closed

Replace shellscirpt with ...? #825

kachick opened this issue Oct 8, 2024 · 5 comments · Fixed by #876

Comments

@kachick
Copy link
Owner

kachick commented Oct 8, 2024

When addressed #741, I felt using pipe is not easy in golang, how about to use https://github.com/bitfield/script for that purpose?

My pain for shell scripts.

  • Parsing and using options
  • Difficult to find reference, it much uses symbols
  • Writing test
  • Nested quoting
  • Will not work in Windows
  • macOS should prefer old bash (using nix reduced this pain)

Benefit of keeping bash

  • I can copy and paste to try the script, and debug, improvements. If using go for wrapper, translating steps should be required

ref: GH-183, GH-281

@kachick kachick added the enhancement New feature or request label Oct 8, 2024
@kachick kachick added this to 🛸 Oct 8, 2024
@github-project-automation github-project-automation bot moved this to 🙋‍♂ in 🛸 Oct 8, 2024
@kachick kachick removed the enhancement New feature or request label Oct 8, 2024
@kachick
Copy link
Owner Author

kachick commented Oct 8, 2024

@kachick
Copy link
Owner Author

kachick commented Oct 9, 2024

kachick added a commit that referenced this issue Oct 22, 2024
* Implement `envs` command to make easier debugging GH-755 and developing GH-853

* Rewrite the command with ruby, the pipeline_w much helps for daily use. So closes GH-825 for now
@github-project-automation github-project-automation bot moved this from 🤷‍♂️ to 🎉 in 🛸 Oct 22, 2024
@kachick
Copy link
Owner Author

kachick commented Nov 11, 2024

@kachick
Copy link
Owner Author

kachick commented Nov 28, 2024

@kachick
Copy link
Owner Author

kachick commented Nov 28, 2024

@kachick kachick changed the title Try bitfield/script for replacing shellscirpt with golang Replace shellscirpt with ...? Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant