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

[CI] Move away from the use of Gulp #6193

Open
chalin opened this issue Feb 4, 2025 · 1 comment
Open

[CI] Move away from the use of Gulp #6193

chalin opened this issue Feb 4, 2025 · 1 comment
Labels

Comments

@chalin
Copy link
Contributor

chalin commented Feb 4, 2025

Roll our own scripts indepentent of Gulp, even for (file) stream processing.

Here are the tasks to roll over:

$ npx gulp --tasks
Tasks for ~/tmp/opentelemetry.io/gulpfile.js
├── default            Display usage instructions
├── lint-md            Check markdownlint rules. For details, use --info.
├── prune              Prune --num <n> entries from static/refcache.json file. For details, use --info.
└── validate-registry

I had originally thought that use of Gulp would help in making our use of markdownlint more efficient. It has, but we can do stream processing in other ways (be if to use OS commands to drive paralleism, or JS-native functionality). Besides, because of the current setup, lint-md is a bit more of a hack; plus it's fix functionality doesn't fully work. So given that I'm going to be rewriting how we use markdownlint (including our own custom rules), it feels like it's a good time to move drop Gulp, the overhead (in learning and maintenance), isn't worth it.

WDYT @svrnm (since you wrote one of the tasks) et all?

@chalin chalin added CI/infra CI & infrastructure cleanup/refactoring e1-hours Effort: < 8 hrs p2-medium labels Feb 4, 2025
@svrnm
Copy link
Member

svrnm commented Feb 4, 2025

Yeah. 👍 when I wrote my task I only plugged it into gulp because it was there already. I would not mind replacing it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants