Skip to content
This repository has been archived by the owner on Nov 17, 2022. It is now read-only.

Solve some problems in use #102

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

is2ac-zhao
Copy link

upgrade depency package version:
requests==2.27.1
WeasyPrint==54.1

fix image cannot display issues:
#54
#72

add github action:
docker build and pull to dockerhub

RUN apt-get install -y locales \
&& dpkg-reconfigure --frontend=noninteractive locales

RUN git clone https://github.com/is2ac-zhao/gitbook2pdf /app
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you use COPY not git clone?

@@ -189,7 +189,7 @@ def __init__(self, base_url, fname=None):
self.meta_list.append(
('generator', 'gitbook2pdf')
)
weasyprint.HTML._ua_stylesheets = local_ua_stylesheets
# weasyprint.HTML._ua_stylesheets = local_ua_stylesheets
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is to ensure a consistent Style, why remove it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants