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

Include schemes with records, when calling finalise_installation #75

Merged
merged 1 commit into from
Oct 13, 2021

Conversation

FFY00
Copy link
Member

@FFY00 FFY00 commented Sep 25, 2021

Signed-off-by: Filipe Laíns [email protected]

@FFY00
Copy link
Member Author

FFY00 commented Sep 25, 2021

This is needed for the bytecode generation step afterwards. This commit is/was part of #66 but I have created a separate PR to speed up review. This is the only thing preventing me from providing a CLI in a 3rd party package.

@FFY00 FFY00 force-pushed the install-return-records branch 4 times, most recently from 4654baf to 13b8204 Compare September 25, 2021 15:56
src/installer/_core.py Outdated Show resolved Hide resolved
@FFY00 FFY00 closed this Sep 27, 2021
@FFY00 FFY00 reopened this Sep 27, 2021
@FFY00 FFY00 force-pushed the install-return-records branch 2 times, most recently from 6306a5c to 9cedef5 Compare September 27, 2021 22:15
@FFY00 FFY00 changed the title core: return written records in install destinations: receive scheme>records mapping in finalize_installation Sep 27, 2021
@FFY00 FFY00 mentioned this pull request Sep 27, 2021
@FFY00 FFY00 force-pushed the install-return-records branch 2 times, most recently from f879826 to 5291351 Compare September 27, 2021 22:50
@FFY00 FFY00 mentioned this pull request Sep 27, 2021
@FFY00 FFY00 requested a review from pradyunsg October 6, 2021 12:37
@FFY00
Copy link
Member Author

FFY00 commented Oct 11, 2021

@pradyunsg friendly ping.

Copy link
Member

@pradyunsg pradyunsg left a comment

Choose a reason for hiding this comment

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

This looks great, although I'd strongly prefer to not have a mapping here.

I'll cut a 0.4.0 with this, once that is changed, that is Python 2 compatible -- only because you're nice and have been super helpful with this project. :)

src/installer/destinations.py Outdated Show resolved Hide resolved
src/installer/_core.py Outdated Show resolved Hide resolved
@FFY00
Copy link
Member Author

FFY00 commented Oct 11, 2021

Thank you, I will have a look tomorrow.

@FFY00
Copy link
Member Author

FFY00 commented Oct 13, 2021

Sorry for the delay.

@pradyunsg pradyunsg changed the title destinations: receive scheme>records mapping in finalize_installation Include schemes with records, when passed into finalise_installation Oct 13, 2021
@pradyunsg pradyunsg changed the title Include schemes with records, when passed into finalise_installation Include schemes with records, when calling finalise_installation Oct 13, 2021
@pradyunsg pradyunsg merged commit 2968e5f into pypa:main Oct 13, 2021
@pradyunsg
Copy link
Member

Sorry for the delay.

No worries! ^>^

Here you go: https://pypi.org/project/installer/0.4.0/

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.

3 participants