-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Docs: Broken Links #792
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
Comments
Good luck! |
@orta I'm trying to get a local copy but not able to install dependency (tried both yarn and npm). Never faced this issue before, so I'm not sure about this. Can you guide me, if you have any idea about it? Node version: v14.5.0 |
It's possible you tried npm before yarn, can you try deleting your node modules and trying again? |
I tried with deleting npm modules and using |
What does the error message say? There doesn't look to be one in your screenshot |
When I do
|
I am getting a similar error.
|
I've submitted a PR which replaces all of the failing deps - for both of your logs. I can't remove sharp, so @alvarocamillont - you'll need to read the logs there to see what's up |
Ok! |
I use a Mac and Windows, and CI runs both linux and windows. So, this repo does cover being built on every major OS for every PR - which is why it's strange that you two are having issues. |
I see ...
What version of node are you using?
…On Wed, Jul 22, 2020 at 8:43 AM Orta Therox ***@***.***> wrote:
I use a Mac and Windows, and CI runs both linux and windows.
So, this repo does cover being built on every major OS for every PR -
which is why it's strange that you two are having issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#792 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AECM4F7WHZNOQVYSUY763WTR43GGXANCNFSM4PCVD7XQ>
.
|
I think CI uses 14 |
Ok!
I am gonna try this version with the new PR.
Thanks Orta!
…On Wed, Jul 22, 2020 at 8:52 AM Orta Therox ***@***.***> wrote:
❯ node -v
v13.6.0
❯ yarn -v
2.1.1
I think CI uses 14
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#792 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AECM4FYLD7S37GPKR4PBLNLR43HG5ANCNFSM4PCVD7XQ>
.
|
OK, that's merged into master now |
I've fixed the related links in the tsconfig - thanks for the issue |
Hi @orta ! |
@orta @alvarocamillont Hi Guys, how did you guys fix this issue please ? markdown-spellcheck@npm:1.3.1 doesn't provide sinon (p2d3db), requested by sinon-as-promised |
Page URL: https://www.typescriptlang.org/tsconfig
Issue: Scroll down to the explanation section, where on the side, you have Default, Release, and Related. There all the Related links are broken. They are supposed to take you to a specific heading section but they redirect you to a new broken page.
Recommended Fix: The fix is pretty easy, just need to modify the paths.
Browser: All Browsers.
NOTE: I actually know how to fix this issue and I would like to try it out, It will be my first contribution to this repo. I am opening an issue in case if I fail to fix it, so I can ask for help and keep the conversation here in this issue.
The text was updated successfully, but these errors were encountered: