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.

Sign up for GitHub

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

Jump to bottom

Scripts directory appears on the website #23

Closed
dlesnoff opened this issue Jun 2, 2023 · 7 comments · Fixed by #24
Closed

Scripts directory appears on the website #23

dlesnoff opened this issue Jun 2, 2023 · 7 comments · Fixed by #24
Labels
bug Something isn't working needs upstream fix Issue depends on the Nim compiler/TheAlgorithms website

Comments

@dlesnoff
Copy link
Collaborator

dlesnoff commented Jun 2, 2023

https://the-algorithms.com/language/nim
The directory.nim script should not appear here, nor any future scripts.
@ZoomRmc
Copy link
Contributor

ZoomRmc commented Jun 2, 2023

Renaming to .scripts should suffice.
Let's push #21 and I can change it everywhere.

@Panquesito7 Panquesito7 added the bug Something isn't working label Jun 2, 2023
@dlesnoff
Copy link
Collaborator Author

dlesnoff commented Jun 2, 2023

I merged your pull request.

@ZoomRmc ZoomRmc mentioned this issue Jun 2, 2023
Merged
@dlesnoff
Copy link
Collaborator Author

dlesnoff commented Jun 3, 2023

Sorry it still appears. We forgot to regenerate the Directory.md file.

@dlesnoff dlesnoff reopened this Jun 3, 2023
@ZoomRmc
Copy link
Contributor

ZoomRmc commented Jun 3, 2023

I think this is just a caching issue. The link for the "Directory" solution is to the moved file.

@ZoomRmc
Copy link
Contributor

ZoomRmc commented Jun 3, 2023

Here's the script that populates the site. I'm not sure if it ignores dotted directories.
https://github.com/TheAlgorithms/website/blob/main/scripts/fetch-algorithms.ts

@dlesnoff
Copy link
Collaborator Author

This issue depends on the following website's issue: TheAlgorithms/website#237

@dlesnoff dlesnoff added the needs upstream fix Issue depends on the Nim compiler/TheAlgorithms website label Jun 20, 2023
@Panquesito7
Copy link
Member

Closing as this is related to the website and not this repository itself.

@Panquesito7 Panquesito7 closed this as not planned Won't fix, can't repro, duplicate, stale Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working needs upstream fix Issue depends on the Nim compiler/TheAlgorithms website
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Hide the scripts dir from the project's site ZoomRmc/TheAlgorithmsNim
3 participants
@ZoomRmc @Panquesito7 @dlesnoff