-
Notifications
You must be signed in to change notification settings - Fork 5.9k
docs: Update some more links #4806
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
Conversation
I will try checking the docs too
Codecov Report
@@ Coverage Diff @@
## main #4806 +/- ##
=======================================
Coverage 69.18% 69.18%
=======================================
Files 29 29
Lines 1652 1652
Branches 363 363
=======================================
Hits 1143 1143
Misses 432 432
Partials 77 77
Continue to review full report at Codecov.
|
Hey, one small doubt - now I edited the package.json and GitHub triggered an action(NPM). Does that mean the base branch's package.json gets overruled? Meaning, does the NPM registry show the new links? |
I just had a chat with @bpmct on Slack - there are a lot of files that got updated. Hope it doesn't meddle with builds. |
/ping @jsjoeio please let me know if I have to change anything. |
/ping @jsjoeio I think the PR is ready for review. |
Also, a question - does FMT mean formatting? Like Prettier code formatting? |
@im-coder-lg thanks for doing this! Yes, ideally you run |
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.
Most of it is good but some places we're accidentally wrote "coderer"
Let's fix those and then we'll be good 👍
docs/CONTRIBUTING.md
Outdated
@@ -61,7 +61,7 @@ These are required by VS Code. See [their Wiki](https://github.com/microsoft/vsc | |||
|
|||
## Creating pull requests | |||
|
|||
Please create a [GitHub Issue](https://github.com/cdr/code-server/issues) that | |||
Please create a [GitHub Issue](https://github.com/coderer/code-server/issues) that |
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.
Please create a [GitHub Issue](https://github.com/coderer/code-server/issues) that | |
Please create a [GitHub Issue](https://github.com/coder/code-server/issues) that |
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.
Never saw those coming! I'll fix it.
docs/CONTRIBUTING.md
Outdated
@@ -78,12 +78,12 @@ we'll guide you. | |||
|
|||
## Development workflow | |||
|
|||
The current development workflow is a bit tricky because we have this repo and we use our `cdr/vscode` fork inside it with [`yarn link`](https://classic.yarnpkg.com/lang/en/docs/cli/link/). | |||
The current development workflow is a bit tricky because we have this repo and we use our `coderer/vscode` fork inside it with [`yarn link`](https://classic.yarnpkg.com/lang/en/docs/cli/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.
The current development workflow is a bit tricky because we have this repo and we use our `coderer/vscode` fork inside it with [`yarn link`](https://classic.yarnpkg.com/lang/en/docs/cli/link/). | |
The current development workflow is a bit tricky because we have this repo and we use our `coder/vscode` fork inside it with [`yarn link`](https://classic.yarnpkg.com/lang/en/docs/cli/link/). |
docs/FAQ.md
Outdated
@@ -302,7 +302,7 @@ after a predetermined amount of time, you can do so by checking continuously for | |||
the last modified time on the heartbeat file. If it is older than X minutes (or | |||
whatever amount of time you'd like), you can kill code-server. | |||
|
|||
Eventually, [#1636](https://github.com/cdr/code-server/issues/1636) will make | |||
Eventually, [#1636](https://github.com/coderer/code-server/issues/1636) will make |
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.
Can we fix this file too?
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.
Done, I'll commit soon.
docs/guide.md
Outdated
@@ -345,8 +345,8 @@ instead and the path will be passed as is (e.g., `/absproxy/3000/my-app-path`). | |||
### Proxying to create a React app | |||
|
|||
You must use `/absproxy/<port>` with `create-react-app` (see | |||
[#2565](https://github.com/cdr/code-server/issues/2565) and | |||
[#2222](https://github.com/cdr/code-server/issues/2222) for more information). | |||
[#2565](https://github.com/coderer/code-server/issues/2565) and |
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.
and this one
docs/helm.md
Outdated
@@ -2,15 +2,15 @@ | |||
|
|||
[](https://img.shields.io/badge/Version-1.0.0-informational?style=flat-square) [](https://img.shields.io/badge/Type-application-informational?style=flat-square) [](https://img.shields.io/badge/AppVersion-4.0.2-informational?style=flat-square) | |||
|
|||
[code-server](https://github.com/cdr/code-server) code-server is VS Code running | |||
[code-server](https://github.com/coderer/code-server) code-server is VS Code running |
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.
and this one
docs/ipad.md
Outdated
@@ -104,26 +104,26 @@ and tricks helpful: | |||
process](#access-code-server-with-a-self-signed-certificate-on-an-ipad) | |||
- Keyboard issues: | |||
- The keyboard disappear sometimes | |||
[#979](https://github.com/cdr/code-server/issues/979) | |||
[#979](https://github.com/coderer/code-server/issues/979) |
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.
and this one
docs/npm.md
Outdated
@@ -27,7 +27,7 @@ which is currently `14.x`. VS Code also [lists Node.js | |||
requirements](https://github.com/microsoft/vscode/wiki/How-to-Contribute#prerequisites). | |||
|
|||
Using other versions of Node.js [may lead to unexpected | |||
behavior](https://github.com/cdr/code-server/issues/1633). | |||
behavior](https://github.com/coderer/code-server/issues/1633). |
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.
and here
@@ -28,8 +28,8 @@ This will show issues that: | |||
1. If more information is required, please ask the submitter and tag as | |||
`waiting-for-info` and wait. | |||
1. Finally, the issue should be moved into the | |||
[code-server](https://github.com/cdr/code-server/projects/1) project where we |
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.
and here
package.json
Outdated
}, | ||
"repository": "https://github.com/cdr/code-server", | ||
"repository": "https://github.com/coderer/code-server", |
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.
and this one
My god, a lot of coderers, I'll fix this. |
I'll change that when I boot my PC. My phone's doing some black magic, it's posting gibberish when I try editing. This will be done by 6:45 AM IST. |
Committing! @jsjoeio I think everythiong's fixed now. |
I should've used all three toggles in the Search/Replace tab in the GItHub.dev editor.
From now, I better be careful with the Find/Replace feature, if I enabled all three toggles, this wouldn't have happened. |
I committed the formatted code. Tried rebasing but everything seemed to be up-to-date, so I did formatting twice, to confirm. |
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.
Love that GIF, that's what I would want to do. Thanks for the opportunity! |
* Update links in package.json I will try checking the docs too * docs: Update links in triage.md * docs: Update links in npm.md * docs: Update links in whatever files that have `cdr` * Replace globally, thanks @bpmct! * fix: coderer instead of coder I should've used all three toggles in the Search/Replace tab in the GItHub.dev editor. * Code Formatting
I will try checking the docs too, that might be easier here :)
Fixes #4802
Don't merge this right now, I'll update the documentation. Sorry for the ping @jawnsy but you only can help me with this - docs preview maybe locally or on the cloud. @jsjoeio I'll inform you once the PR is ready, should I do rebasing and FMTs if more commits are added to the HEAD branch?
Again, sorry for the pings, I want to finish this soon.