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

Valloric / ycmd / 1197
94%

Build:
DEFAULT BRANCH: master
Ran 24 Jan 2016 11:55PM UTC
Jobs 1
Files 42
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

pending completion
1197

push

travis-ci

homu
Auto merge of #311 - puremourning:tern-refactor-rename-upstream, r=Valloric

[READY] Support for rename in tern completer

## Tern rename

Tern server supports renames of variables only. It can do this across multiple files, only when it knows about those files. This feature is supported by tern_for_vim, so with the addition of this feature we should have feature-parity on that front.

## Refactoring infrastructure

This re-uses the existing FixIt infrastructure, with no changes to the JSON API. The only change is that this is the first time the command will return changes in multiple files.

The largest change is moving the Cs* classes that are equivalent of the C++ classes representing diagnostics/FixIts to `responses.py`, so they can be shared by other completers.

A Vim client change is required to recognise FixIt responses by the contents of the response, rather than the command name, and to apply the chunks across files.

## FileReadyToParse

Previously, the server would only learn about files if we sent a request (such as a completion request), or by appearing in the loadEagerly list.  Now we pass the file data to the server in the parse request.  This ensures that the server knows about files as soon as possible, so that they are available for completion suggestions and for rename/etc. commands.

## Tests

Simple tests for the single- and multi-file rename cases, both eagerly loaded and picked up from parse request.

## JSON API Changes

None.

## Client changes required

* FixIt commands are now identified by the presence of `'fixit'` entry in the  response
* FixIt commands can now apply across multiple files. The client is responsible for applying the chunks in an appropriate fashion.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/311)
<!-- Reviewable:end -->

46 of 46 new or added lines in 3 files covered. (100.0%)

2597 of 3142 relevant lines covered (82.65%)

0.83 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
3
100.0
ycmd/completers/javascript/tern_completer.py
Jobs
ID Job ID Ran Files Coverage
5 1197.5 (USE_CLANG_COMPLETER=true YCMD_PYTHON_VERSION=2.7 COVERAGE=true) 24 Jan 2016 11:55PM UTC 0
82.65
Travis Job 1197.5
Source Files on build 1197
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #1197
  • d0b2e567 on github
  • Prev Build on master (#1192)
  • Next Build on master (#1200)
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