Skip to content

re-import book images from progit2 repo #952

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

Merged
merged 1 commit into from
Feb 10, 2017
Merged

re-import book images from progit2 repo #952

merged 1 commit into from
Feb 10, 2017

Conversation

peff
Copy link
Member

@peff peff commented Feb 10, 2017

We did this originally in #775, but the book hierarchy changed to a flat one since then. The new images are missing some updates and fixes.

I've confirmed from the diff that this fixes some issues (e.g., check out advance-master.png which fixes the flipped commit sha1s).

The new images have lower resolution, but I think that's fine. That comes from progit/progit2@3de8ef2, and that's what's been going into the book builds for a long time.

/cc @jnavila @pedrorijo91

We did this originally in #775, but the book hierarchy
changed to a flat one since then. The new images are missing
some updates and fixes.
@peff
Copy link
Member Author

peff commented Feb 10, 2017

Oh, and I added a short shell script to do the import. It depends on having a progit2 checkout nearby, but I think it's OK for it to be picky. This should only need to be run very occasionally.

@peff
Copy link
Member Author

peff commented Feb 10, 2017

I deployed this to the site and confirmed that #515 is now fixed. This should refix the other image problems, too.

@peff peff merged commit e16ec9b into master Feb 10, 2017
@peff peff deleted the import-images branch February 10, 2017 22:17
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