Refactor directory structure, update _config.yml to match opensource.guide#1283
Refactor directory structure, update _config.yml to match opensource.guide#1283
Conversation
…guide This change updates the directory structure of the jekyll site, moving the language-specific directories under `_articles` and updating the front matter / `_config.yml` to match. The goal was to clean up the root of the repository and bring the config in line with that used by the opensource.guide site.
There was a problem hiding this comment.
Copilot reviewed 48 out of 54 changed files in this pull request and generated no comments.
Files not reviewed (6)
- _articles/ar/index.html: Language not supported
- _articles/bg/index.html: Language not supported
- _articles/bn/index.html: Language not supported
- _articles/de/index.html: Language not supported
- _articles/es/index.html: Language not supported
- _articles/fa/index.html: Language not supported
Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more
jmeridth
left a comment
There was a problem hiding this comment.
All LGTM. Very nice.
Non blocking question:
Do we want the index.html, businesses.html and maintainers.html now in the _articles root to be under an en directory or that will break things?
I think without more extensive surgery they need to live where they are. Based on my primitive understanding, this is because they are the default |
That's what I guessed. Had to ask. Nice clean up and standardize. |
This change updates the directory structure of the jekyll site, moving
the language-specific directories under
_articlesand updating thefront matter /
_config.ymlto match.The goal was to clean up the root of the repository and bring the
config in line with that used by the opensource.guide site.
Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.