Skip to content

Conversation

@pfitzseb
Copy link
Member

@pfitzseb pfitzseb commented Nov 6, 2020

No description provided.

@aviatesk
Copy link
Member

aviatesk commented Nov 6, 2020

nice, I will read their idea and revisit this PR this weekend

Copy link
Member

@aviatesk aviatesk left a comment

Choose a reason for hiding this comment

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

Nice work ! Added few inline comments, but I don't have any strong opinion on them so it's totally okay to me if you merge this PR as is.

* The language server process is now started with the `JULIA_LANGUAGESERVER` environment variable set to `1` ([#1707](https://github.com/julia-vscode/julia-vscode/pull/1707)).
* Added commands to re-start the LS or re-index the symbol cache ([#1721](https://github.com/julia-vscode/julia-vscode/pull/1721)).
* `@edit` now works properly on [code-server](https://github.com/cdr/code-server) instances ([#1737](https://github.com/julia-vscode/julia-vscode/pull/1737)).
* Added commands to `cd` to the current directory, `Pkg.activate` the current directory, or `Pkg.activate` the current files nearest project. These commands are also available in the file explorer ([#1743](https://github.com/julia-vscode/julia-vscode/pull/1743)).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Added commands to `cd` to the current directory, `Pkg.activate` the current directory, or `Pkg.activate` the current files nearest project. These commands are also available in the file explorer ([#1743](https://github.com/julia-vscode/julia-vscode/pull/1743)).
* Added commands to `cd` to the current directory (`Julia: Change to This Directory`), `Pkg.activate` the current directory (`Julia: Activate This Environment`), or `Pkg.activate` the project nearest the current files (`Julia: Activate Parent Environment`)). These commands are also available in the file explorer ([#1743](https://github.com/julia-vscode/julia-vscode/pull/1743)).

May look rather verbose though.

pfitzseb and others added 3 commits November 10, 2020 14:46
Co-authored-by: Shuhei Kadowaki <40514306+aviatesk@users.noreply.github.com>
Co-authored-by: Shuhei Kadowaki <40514306+aviatesk@users.noreply.github.com>
Co-authored-by: Shuhei Kadowaki <40514306+aviatesk@users.noreply.github.com>
@aviatesk aviatesk merged commit fcf207a into master Nov 10, 2020
@aviatesk aviatesk deleted the sp/keepachangelog branch November 10, 2020 14:42
@oppo-source oppo-source removed the request for review from a team April 16, 2021 07:35
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.

3 participants