Add links to project structure documentation #56
Add links to project structure documentation #56
Conversation
Hi @JasonAizkalns thanks for the PR! It appears some of the links you've added have stray spaces in them, e.g., |
Thank you @reubano. Let me know if cleaning the whitespace and hanging parentheses changes things for the MD formatted links. Interestingly enough, when I use atom and the markdown preview package things looks fine: If that still doesn't work, I'll try the inline html approach. Aside, do you know where to find the PR template? It's mentioned in a few places, but I have not come across the appropriate formatting? Finally, please note that I am still getting used to contributing to repos that are not my own, so I apologize for what's likely a messy commit (the |
Hm, this PR/commit seems to add a bunch of git-lfs related files which appears to be a mistake (or is not otherwise documented...) |
That is weird, maybe it's an issue with GHFMD. |
hmm, looks like commit #ed1693f00 is the culprit. Try a |
@JasonAizkalns Can you rebase this branch and remove the .dcm / git-lfs changes? :) |
@lamby any chance you can verify that the links look good in the build? |
They don't render here for me, alas. Dumb question, but can't we just use |
@JasonAizkalns I see you pushed a change to move to How come we went with the markdown ones in the first place? Just curious how you tested this? :) |
@JasonAizkalns Friendly ping on this? :) |
Sure, I'll reopen the old pull request. Are there any changes you want to add to the current commit? |
@eltonlaw wrote:
Not that I can immediately see, no :) Thanks! |
Hey all, what's this blocking on? Should I be looking for another commit/PR/issue? :) Best wishes, — Chris |
Chris Lamb wrote:
@eltonlaw @JasonAizkalns @reubano ^^ ;) |
@reubano Okay, am assigning to you. :) |
superseded by #83 |
Description
Adds links to the project structure documentation.
Reference to official issue
Addresses issue #15
Motivation and Context
Turns the references to the technologies into useful links to those technologies.
How Has This Been Tested?
Clicked on links.
CLA