-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Conversation
One correction, I did scan other files as well, and thus ended up touching As to the build failure, I'm uncertain how this PR can cause any build behaviour change from the previous run. I'm new to the flux repo, and would love if anyone can lend a hand looking into it. |
@@ -556,7 +556,7 @@ Commit customization | |||
--git-user | |||
--git-email | |||
|
|||
See [docs/daemon.md] for more information. | |||
See [docs/features/daemon.md] for more information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe let's make this a real clickable link?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! 🍍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for your first contribution @rytswd 🌻
After #2152 has been merged, it looks like some of the doc had broken links to other docs due to the path changes. I just scanned all the
*.md
files that had some directory structure changes in that PR, and updated the entire repo with string replace.I think I covered most use cases, but please shout if I missed / wrongly replaced any.