Skip to content

Added script for automatically generating DIRECTORY.md#889

Merged
poyea merged 3 commits into
TheAlgorithms:masterfrom
NielXu:master
Jun 17, 2019
Merged

Added script for automatically generating DIRECTORY.md#889
poyea merged 3 commits into
TheAlgorithms:masterfrom
NielXu:master

Conversation

@NielXu

@NielXu NielXu commented Jun 8, 2019

Copy link
Copy Markdown
Contributor

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

NielXu commented Jun 8, 2019

Copy link
Copy Markdown
Contributor Author

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

@cclauss

cclauss commented Jun 8, 2019

Copy link
Copy Markdown
Member

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

@cclauss

cclauss commented Jun 8, 2019

Copy link
Copy Markdown
Member

#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

NielXu commented Jun 8, 2019

Copy link
Copy Markdown
Contributor Author

@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

cclauss commented Jun 9, 2019

Copy link
Copy Markdown
Member

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

@poyea

poyea commented Jun 17, 2019

Copy link
Copy Markdown
Member

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

@poyea

poyea commented Jun 17, 2019

Copy link
Copy Markdown
Member

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