-
Notifications
You must be signed in to change notification settings - Fork 590
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
doc/adr vs. docs/adr #39
Comments
You can use whatever directory you like, when you call |
I would, however, change the default path to This hits this repo itself: https://github.com/npryce/adr-tools/tree/master/doc/adr would be renamed to docs/adr. Additionally, the README and code: |
This would be a backwards incompatible change. Every current user would have to configure their project to point at their existing directory. |
@npryce I read that answer that there will be no 3.0.0 version supporting this directly. I will try to propose a patch checking the default directory for |
GitHub pages allows for publishing websites on github. As source, they have following sources
gh-pages
branch, folder/
master
branch, folder/
master
branch, folder/docs
. - https://help.github.com/articles/configuring-a-publishing-source-for-github-pages/#publishing-your-github-pages-site-from-a-docs-folder-on-your-master-branchNote the
s
in the folder name.In the (unrelated) project Eclipse Winery, the homepage is rendered from
docs/
. The ADRs are placed into the folderdocs/adr
. As a consequence, the ADRs are browsable in the homepage: http://eclipse.github.io/winery/adr/In case the ADRs resided in
doc/adr
, there would be two folders,docs/
for the whole homepage anddoc/adr
for all the ADRs.Therefore, I propose to rename the folder
doc
todocs
. I would submit a pull request, but seeing a nice test coverage (refs #30 (comment)), I would only start if this is a consensus to rename the directory.The text was updated successfully, but these errors were encountered: