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 |
jsjoeio
left a comment
There was a problem hiding this comment.
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
| ## 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.
| 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.
Never saw those coming! I'll fix it.
docs/CONTRIBUTING.md
Outdated
| ## 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.
| 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
| 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.
Done, I'll commit soon.
docs/guide.md
Outdated
| 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 |
docs/helm.md
Outdated
| [](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 |
docs/ipad.md
Outdated
| - Keyboard issues: | ||
| - The keyboard disappear sometimes | ||
| [#979](https://github.com/cdr/code-server/issues/979) | ||
| [#979](https://github.com/coderer/code-server/issues/979) |
docs/npm.md
Outdated
|
|
||
| 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). |
| 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 |
package.json
Outdated
| "url": "https://github.com/coderer/code-server/issues" | ||
| }, | ||
| "repository": "https://github.com/cdr/code-server", | ||
| "repository": "https://github.com/coderer/code-server", |
|
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. |
|
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.