Skip to content

Build DIRECTORY.md with JavaScript, not Python #160

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

Closed
cclauss opened this issue May 9, 2020 · 3 comments · Fixed by #167
Closed

Build DIRECTORY.md with JavaScript, not Python #160

cclauss opened this issue May 9, 2020 · 3 comments · Fixed by #167
Labels
feature Adds a new feature help wanted Needs to be worked on

Comments

@cclauss
Copy link
Member

cclauss commented May 9, 2020

We present all of of our algorithms in an organized list, DIRECTORY.md so visitors can rapidly find topics of interest. Currently DIRECTORY.md is being autogenerated by one of our GitHub Actions which uses Python code to create the directory text.

Why should the JavaScript repo rely on Python code?!? Especially given that GitHub Actions fully support the creation of JavaScript-based Actions. Can we use JavaScript to autogenerate DIRECTORY.md and lose the Python code in the process?

@itsvinayak itsvinayak added help wanted Needs to be worked on feature Adds a new feature labels May 9, 2020
@sahilbansal17
Copy link
Contributor

@cclauss I would like to work on this.

@cclauss
Copy link
Member Author

cclauss commented May 9, 2020

No need to wait for permission... Just do it!

@sahilbansal17
Copy link
Contributor

I have added the workflow as a js file in PR #163 .
The yml file needs to be updated so that the action can be run, I tried to look up github actions on javascript documentation and try to figure it out but didn't understand how to run the standalone js file. Some help will be appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adds a new feature help wanted Needs to be worked on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants