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

Gftools packager: several issues #239

Merged
merged 7 commits into from
Sep 24, 2020

Conversation

graphicore
Copy link
Contributor

@graphicore graphicore commented Aug 19, 2020

See #238: trying to fix wrongly selected remote for PR

Fixes an issue that happened to me when testing this, unlikely in normal usage though

I don't know if _pygit2.GitError: unsupported URL protocol is fixed with this!

UPDATE:

also fix #233 remove redundant info from upstream.yaml, remove comments from upstream.yaml

@graphicore graphicore requested a review from m4rc1e August 19, 2020 17:11
@graphicore graphicore changed the title Gftools packager: fix a bug when selecting the remote for making a PR Gftools packager: several issues Aug 20, 2020
Copy link
Collaborator

@m4rc1e m4rc1e left a comment

Choose a reason for hiding this comment

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

Thank you for adding the OS X fix. We need to make another one for when we fetch the remote. I'll comment on the exact line now.

Lib/gftools/packager.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@m4rc1e m4rc1e left a comment

Choose a reason for hiding this comment

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

LGTM. lmk if you want me to merge and I'll tag a new release for pypi.

@m4rc1e m4rc1e merged commit 0bd3428 into googlefonts:master Sep 24, 2020
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.

Slim down upstream.yaml files?
2 participants