Skip to content

Added script for automatically generating DIRECTORY.md #889

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

Merged
merged 3 commits into from
Jun 17, 2019

Conversation

NielXu
Copy link
Contributor

@NielXu NielXu commented Jun 8, 2019

I added a python script that scans through the directory and generates the corresponding DIRECTORY.md. I have also done some formatting for the names, for example, replace underscores to whitespaces, capitalize folder names. However, contributors must still run python script.py in order to generate DIRECTORY.md.

@NielXu
Copy link
Contributor Author

NielXu commented Jun 8, 2019

And also, I did not check all the links in DIRECTORY.md, I only check some of them and they are all correct.

@cclauss
Copy link
Member

cclauss commented Jun 8, 2019

Slick... Well done. A nice improvement would be sorting of the sub-directories & files.

@cclauss
Copy link
Member

cclauss commented Jun 8, 2019

#890 demonstrates where to put it into .travis.yml but then we need to commit the changed file to the repo under test. Output: https://travis-ci.com/cclauss/Python-3/builds/114846147#L8552

@NielXu
Copy link
Contributor Author

NielXu commented Jun 8, 2019

@cclauss Do you mean sorting files and directories by alphabetical order? I have done sorting the directories and it is not difficult to sort files as well. And thanks I will check out the .travis.yml

@cclauss
Copy link
Member

cclauss commented Jun 9, 2019

Yes. Sorting file directories by alphabetical order too. Thanks much.

@poyea
Copy link
Member

poyea commented Jun 17, 2019

Thank you very much. I'll merge this. It's okay to improve the script or method based on this PR.

@poyea
Copy link
Member

poyea commented Jun 17, 2019

That's a lot of work. Thanks.

@poyea poyea merged commit ef14748 into TheAlgorithms:master Jun 17, 2019
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 this pull request may close these issues.

3 participants