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

Initialize the repository inside the script. #14

Merged
merged 3 commits into from
Apr 6, 2020

Conversation

jyasskin
Copy link
Collaborator

@jyasskin jyasskin commented Apr 2, 2020

Instead of printing shell commands for the user to run.

This also avoids adding files to the initial working directory, which should make it easier to pip-ify these scripts.

It would be possible to use https://gitpython.readthedocs.io/ for a Python interface to Git, but since some of the initialization commands need to be subprocesses anyway, it seemed more straightforward to just call out to the Git utility.

jyasskin added 2 commits April 2, 2020 15:50
Instead of printing shell commands for the user to run.

This also avoids adding files to the initial working directory, which should make it easier to pip-ify these scripts.
@richsalz
Copy link
Owner

richsalz commented Apr 3, 2020

A tour de force.

@jyasskin jyasskin marked this pull request as ready for review April 3, 2020 16:33
@richsalz richsalz self-requested a review April 6, 2020 16:02
@richsalz
Copy link
Owner

richsalz commented Apr 6, 2020

I can live with splitting imports into one line :)
Approved.
One of my issues is to refactor more of this into utility modules, so this is a start.

@jyasskin jyasskin merged commit 42023ff into richsalz:master Apr 6, 2020
@jyasskin jyasskin deleted the init-internally branch April 6, 2020 16:53
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