Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename JS bundles to match corresponding source directory #633

Merged
merged 1 commit into from
Dec 19, 2017

Conversation

robertknight
Copy link
Member

This is part of a series of PRs to address some low hanging fruit to hopefully make it easier for newcomers to find their way around the client code.

Rename the "injector" and "app" bundles to "annotator" and "sidebar"
respectively to match the source directory which produces those bundles.

@codecov
Copy link

codecov bot commented Dec 11, 2017

Codecov Report

Merging #633 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #633   +/-   ##
=======================================
  Coverage   90.98%   90.98%           
=======================================
  Files         135      135           
  Lines        5367     5367           
  Branches      930      930           
=======================================
  Hits         4883     4883           
  Misses        484      484
Impacted Files Coverage Δ
src/boot/boot.js 100% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f1bcb9a...7c0567d. Read the comment docs.

robertknight added a commit that referenced this pull request Dec 11, 2017
Rename the entry point for the annotator CSS bundle so that it matches
the name of the source directory from which the bundle is compiled and
also the name of the corresponding JS bundle once that is renamed by
#633
@robertknight
Copy link
Member Author

Rebased

Rename the "injector" and "app" bundles to "annotator" and "sidebar"
respectively to match the source directory which produces those bundles.
@sheetaluk sheetaluk merged commit 02a4367 into master Dec 19, 2017
@sheetaluk sheetaluk deleted the rename-js-bundles branch December 19, 2017 15:08
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.

2 participants