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

Update globals path (fix pressbooks/pressbooks-book#307) #1452

Merged
merged 14 commits into from Nov 7, 2018

Conversation

greatislander
Copy link
Contributor

@greatislander greatislander commented Nov 6, 2018

This PR adds support for using Buckram in its new home at /pressbooks-book/packages/buckram (see pressbooks/pressbooks-book#307), and updates related functions for normalizing CSS paths and loading Buckram image assets. In addition, since we don't use the EpubCheck apt package anywhere else (as it is a version that lacks the security update in 4.0.2), I switched to installing EpubCheck's latest jarfile. This also changes the default value of the PB_EPUBCHECK_COMMAND constant to use the jarfile.

@greatislander greatislander added this to the 5.6.0 milestone Nov 6, 2018
@greatislander greatislander self-assigned this Nov 6, 2018
@codecov
Copy link

codecov bot commented Nov 7, 2018

Codecov Report

Merging #1452 into dev will increase coverage by <.01%.
The diff coverage is 84.21%.

@@             Coverage Diff              @@
##                dev    #1452      +/-   ##
============================================
+ Coverage     62.48%   62.49%   +<.01%     
- Complexity     4332     4336       +4     
============================================
  Files           114      114              
  Lines         18915    18924       +9     
============================================
+ Hits          11819    11826       +7     
- Misses         7096     7098       +2

@greatislander greatislander merged commit 2397c25 into dev Nov 7, 2018
@greatislander greatislander deleted the globals-path branch November 7, 2018 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants