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

Migrated files from charlies PR #113

Closed
wants to merge 9 commits into from
Closed

Migrated files from charlies PR #113

wants to merge 9 commits into from

Conversation

matentzn
Copy link
Collaborator

@matentzn matentzn commented Aug 23, 2021

This is the exact same pull request as #90

it was erroneously created because of my inability to work with a forked repo i didn't have write rights to - and which had conflicts with master. Now I know better :)

@matentzn
Copy link
Collaborator Author

Merged all of @cthoyt s commits in. @hrshdhgd yours!

@cthoyt
Copy link
Member

cthoyt commented Aug 23, 2021

Still not exactly sure what's different here from my PR... could you update the description please?

@matentzn
Copy link
Collaborator Author

There is absolutely no difference.. Sorry this was a bit of a communication error; we were in a meeting and wanted to quickly work on the branch - no changes to your work (apart from 2 or 3 dependencies in setup.cfg). If you want us to redo this from scratch to be more transparent then we can!

@matentzn
Copy link
Collaborator Author

Closing this now, superseded by #114

@matentzn matentzn closed this Aug 23, 2021
@hrshdhgd hrshdhgd deleted the installation branch August 23, 2021 15:44
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