- Start separating a "standalone" CLI. This is a little awkward with clap-
derive, but I got it working. Detection of whether the CLI _is_
standalone is still todo.
- Remove the old ad-hoc update code for code-server, and use the update
service instead.
- Fix some of the "permission denied" errors people got while updating
before. We need to rename the old running binary, not just overwrite it.
* perf - concatenate windows main files
* Revert "Revert "Use `esbuild` to bundle some CommonJS main files (#160957)" (#161118)"
This reverts commit 84c46b71a5.
* build - exclude server main files
* build - make concat a task that runs like the others
* some renames
* Avoid overwriting the nodejs closure require
* Revert "build - exclude server main files"
This reverts commit 736516624e.
Co-authored-by: Alex Dima <alexdima@microsoft.com>
Resubmission of #157532 with the following changes:
- Use `eslint-plugin-local` instead of `yarn` link to run our plugins
- Move our plugins to a top level `.eslintplugin` dir (as required by `eslint-plugin-local`)
- Update all names to `local/`
* Run our custom eslint rules using ts-node
Use `ts-node` to run our custom eslint rules. This lets us delete the pre-compiled js. It also means you can don't have to compile the rules while editing them
As part of this change, I've also switched us to using an eslint plugin instead of a rulesDir. This is now the preferred way to ship custom rules
* Fix two more disables
* Move ts-node to project root
* Enable transpileOnly
* Add getPullRequestTemplates method to discover templates
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Add method to quick pick for PR templates
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Handle possible PR templates
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Remove unnecessary return value assignment
Co-authored-by: João Moreno <mail@joaomoreno.com>
* Change comparison operands' order
Co-authored-by: João Moreno <mail@joaomoreno.com>
* Remove sorting template URIs in pickPullRequestTemplate
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Sort template URIs before showing quick-pick list
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Rename getPullRequestTemplates method to findPullRequestTemplates
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Find Github PR templates in-parallel using readdir/stat
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Export method for visibitliy in tests
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Add tests for Github PR template detection
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Add launcher configration to run Github tests
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* 💄
* Replace stat with readDirectory for OS native case sensitivity
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Delete some files to avoid duplicate names on case insensitive envs
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
* Exclude deleted files from test case expected result
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
Co-authored-by: João Moreno <mail@joaomoreno.com>
Co-authored-by: João Moreno <joao.moreno@microsoft.com>
* web - first cut `yarn web` via our server
* properly pipe output
* web - remove traces of web playground
* web - remember last opened workspace for convinience
* use vscode-test-web for server less, clean up web commands
* fix comment
* fix `yarn web`
* rename to code-server
* open system browser
* code-server script: use minimist
* test resolver: use ./scripts/code-server
* integartion tests: fix code-server command name
Co-authored-by: Martin Aeschlimann <martinae@microsoft.com>