Skip to content

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

Merged
merged 7 commits into from
Feb 1, 2022
Merged

docs: Update some more links #4806

merged 7 commits into from
Feb 1, 2022

Conversation

im-coder-lg
Copy link
Contributor

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.

I will try checking the docs too
@im-coder-lg im-coder-lg requested a review from a team January 29, 2022 03:27
@codecov
Copy link

codecov bot commented Jan 29, 2022

Codecov Report

Merging #4806 (1d268b4) into main (439f27c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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           
Impacted Files Coverage Δ
src/node/cli.ts 83.58% <ø> (ø)
src/node/coder_cloud.ts 27.27% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 439f27c...1d268b4. Read the comment docs.

@im-coder-lg im-coder-lg changed the title Fix more links docs: Update some more links Jan 29, 2022
@im-coder-lg
Copy link
Contributor Author

im-coder-lg commented Jan 29, 2022

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?

@im-coder-lg
Copy link
Contributor Author

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.

@im-coder-lg
Copy link
Contributor Author

/ping @jsjoeio please let me know if I have to change anything.

@im-coder-lg
Copy link
Contributor Author

/ping @jsjoeio I think the PR is ready for review.

@im-coder-lg
Copy link
Contributor Author

Also, a question - does FMT mean formatting? Like Prettier code formatting?

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 31, 2022

@im-coder-lg thanks for doing this! Yes, ideally you run yarn fmt to fix the formatting then commit and push up. Should be good after that.

@jsjoeio jsjoeio self-assigned this Jan 31, 2022
@jsjoeio jsjoeio added this to the 4.0.3 milestone Jan 31, 2022
Copy link
Contributor

@jsjoeio jsjoeio left a 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 👍

@@ -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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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

Copy link
Contributor Author

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.

@@ -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/).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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
Copy link
Contributor

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?

Copy link
Contributor Author

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
Copy link
Contributor

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 @@

[![Version: 1.0.0](https://img.shields.io/badge/Version-1.0.0-informational?style=flat-square)](https://img.shields.io/badge/Version-1.0.0-informational?style=flat-square) [![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square)](https://img.shields.io/badge/Type-application-informational?style=flat-square) [![AppVersion: 4.0.2](https://img.shields.io/badge/AppVersion-4.0.2-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
Copy link
Contributor

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)
Copy link
Contributor

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).
Copy link
Contributor

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
Copy link
Contributor

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",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and this one

@im-coder-lg
Copy link
Contributor Author

My god, a lot of coderers, I'll fix this.

@im-coder-lg
Copy link
Contributor Author

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.

@im-coder-lg
Copy link
Contributor Author

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.
@im-coder-lg
Copy link
Contributor Author

From now, I better be careful with the Find/Replace feature, if I enabled all three toggles, this wouldn't have happened.

@im-coder-lg
Copy link
Contributor Author

I committed the formatted code. Tried rebasing but everything seemed to be up-to-date, so I did formatting twice, to confirm.

Copy link
Contributor

@jsjoeio jsjoeio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jsjoeio jsjoeio merged commit 10f57ba into coder:main Feb 1, 2022
@im-coder-lg
Copy link
Contributor Author

Love that GIF, that's what I would want to do. Thanks for the opportunity!

TinLe pushed a commit to TinLe/code-server that referenced this pull request Apr 23, 2022
* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Fix links in package.json
2 participants