Skip to content

Commit

Permalink
fix: allow lsp to run inside of a docker container (#3876)
Browse files Browse the repository at this point in the history
# Description

## Problem\*

Resolves <!-- Link to GitHub Issue -->

## Summary\*

This PR re-adds a change which was removed from #3875 which is required
for the LSP to run inside of a docker container.

## Additional Context



## Documentation\*

Check one:
- [x] No documentation needed.
- [ ] Documentation included in this PR.
- [ ] **[Exceptional Case]** Documentation to be submitted in a separate
PR.

# PR Checklist\*

- [x] I have tested the changes locally.
- [x] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
  • Loading branch information
TomAFrench authored Dec 19, 2023
1 parent 7a4e845 commit 2529977
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions tooling/nargo_cli/src/cli/lsp_cmd.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
use async_lsp::{
client_monitor::ClientProcessMonitorLayer, concurrency::ConcurrencyLayer,
panic::CatchUnwindLayer, server::LifecycleLayer, tracing::TracingLayer,
concurrency::ConcurrencyLayer, panic::CatchUnwindLayer, server::LifecycleLayer,
tracing::TracingLayer,
};
use bn254_blackbox_solver::Bn254BlackBoxSolver;
use clap::Args;
Expand Down Expand Up @@ -39,7 +39,6 @@ pub(crate) fn run(
.layer(LifecycleLayer::default())
.layer(CatchUnwindLayer::default())
.layer(ConcurrencyLayer::default())
.layer(ClientProcessMonitorLayer::new(client))
.service(router)
});

Expand Down

0 comments on commit 2529977

Please sign in to comment.