hypothesis / h / 5635
57%
master: 97%

LAST BUILD BRANCH: realtime-update-v1-h-only-1
DEFAULT BRANCH: master
Ran
Jobs 1
Files 60
Run time 36s
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
5635

push

travis-ci

nickstenning 
Improve packaging, bundling and module boilerplate

Prevent negative interactions with module systems present in injected
pages and refactor browserify interactions with angular to increase
CommonJS accessibility / legibility and isolate angular module system
boilerplate.

- Avoid issues due to mis-detected module environments

  - In the injection script, load only Annotator and jQuery in the
    global scope and then both removed with `noConflict()`. Both are
    loaded with browserify-shim, which prevents jQuery from registering
    itself against require.js, if present (fix #2026).

  - Be explicit about registration of common cross-frame components
    (fix #2090).

    - Lift angular glue into app.coffee. Individual components are
      exported as CommonJS modules.

    - Lift Annotator glue into bootstrap.js (renamed hypothesis.js).

- Break up all modules that contained multiple services/directives/etc,
  such as the formHelpers and uiHelpers modules, as well as catch-all
  files like services.coffee and directives.coffee. For a summary of
  these movements see the bottom of this commit message.

- Rename all services, dropping the -service suffix, to match their
  injected names. This is consistent with how directives are being
  treated, and controllers as well (controllers are registered and
  injected done with the "Controller" suffix).

- Move Annotator subclasses into the annotator subdirectory. This
  change avoids having `Annotator.Host` clash with the `host`
  service and makes it easy to glob dependencies in assets.yaml to
  minimally rebuild either the app or the inject.

---

Summary of structural reorganisation. Modules that have been trivially
renamed (from "foo-service" to "foo") are not included in this list:

    controllers:AppController                   app-controller
    controllers:AnnotationViewerController      annotation-controller
    controllers:AnnotationUIController          annotation-ui-controller
    controller... (continued)

1413 of 2492 relevant lines covered (56.7%)

0.57 hits per line

Jobs
ID Job ID Ran Files Coverage
1 5635.1 (BROWSERIFY_PIPE_BIN=bin/browserify-pipe) 0
56.7
 Travis Job 5635.1
Source Files on build 5635
Detailed source file information is not available for this build.