• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

tomv564 / LSP / 1869 / 1
38%
master: 38%

Build:
DEFAULT BRANCH: master
Ran 16 Nov 2019 07:59PM UTC
Files 44
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

16 Nov 2019 07:57PM UTC coverage: 35.607% (+0.9%) from 34.731%
1869.1

push

travis-ci

tomv564
Teach LSP about WorkspaceFolder (#770)

* Teach LSP about workspaces

Introduce the Workspace class in protocol.py. It has a name and a URI.
Refactor most of plugin/core/workspace.py to work with the new Workspace
class.

In the WindowManager class, use the new Workspace class at the boundary
of the session starter callable.

So, the Session class doesn't yet understand what a workspace is, but
the WindowManager does.

This diff does absolutely nothing feature-wise, it just refactors code.

* Remove get_common_prefix, store path instead of URI

* Rename Workspace to WorkspaceFolder

* Use .folders() instead of .project_data() to determine path and name for WorkspaceFolder

* Fix mypy errors hopefully

* Fix pytest tests

* Allow a Window to be passed to maybe_get_workspace_from_view

The maybe_get_workspace_from_view function will check if it's dealing
with a window or a view. If it's dealing with a window, it will use
the active_view() of that window.

* 'type: ignore' is not needed

* Remove unused import

1441 of 4047 relevant lines covered (35.61%)

0.36 hits per line

Source Files on job 1869.1
  • Tree
  • List 0
  • Changed 3
  • Source Changed 3
  • Coverage Changed 3
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 828
  • Travis Job 1869.1
  • f9c68aa9 on github
  • Prev Job for on master (#1860.1)
  • Next Job for on master (#1878.1)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc