You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Mar 25, 2019. It is now read-only.
The README.md asks contributors to run yarn run doc after making changes, but this produces URI links ("Defined In") to their own forks.
This means that:
These files change for every PR by a different contributor, polluting code diffs.
These paths to forks get merged into the upstream repo still containing links to the fork, thereby making them incorrect.
These URI paths should either be relative, or always point to the upstream repo. If this is not possible, they should not be included at all, or generated documentation should not be checked in to the repo.
Example: *Defined in [interface.ts:41](https://github.com/ColinEberhardt/API/blob/eac0696/src/interface.ts#L41)*
The text was updated successfully, but these errors were encountered:
@nkolba Changing the URLs themselves aren't good enough, you need to change the typedoc process, otherwise it is a problem for every PR. Please see my PR #32.
On Sep 24, 2018, at 6:04 AM, Riko Eksteen ***@***.***> wrote:
@nkolba Changing the URLs themselves aren't good enough, you need to change the typedoc process, otherwise it is a problem for every PR. Please see my PR #32.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
The README.md asks contributors to run
yarn run doc
after making changes, but this produces URI links ("Defined In") to their own forks.This means that:
These URI paths should either be relative, or always point to the upstream repo. If this is not possible, they should not be included at all, or generated documentation should not be checked in to the repo.
Example:
*Defined in [interface.ts:41](https://github.com/ColinEberhardt/API/blob/eac0696/src/interface.ts#L41)*
The text was updated successfully, but these errors were encountered: