Skip to content

PyInstaller: add hooks for google.cloud and google.cloud.storage#2637

Merged
htgoebel merged 1 commit intopyinstaller:developfrom
efiop:develop
Jun 3, 2017
Merged

PyInstaller: add hooks for google.cloud and google.cloud.storage#2637
htgoebel merged 1 commit intopyinstaller:developfrom
efiop:develop

Conversation

@efiop
Copy link
Copy Markdown
Contributor

@efiop efiop commented Jun 1, 2017

Copy link
Copy Markdown
Member

@htgoebel htgoebel left a comment

Choose a reason for hiding this comment

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

Thanks for this pull-request. Please add the copyright header which you can find in any other hook file.

@htgoebel htgoebel added the area:hooks Caused by or effecting some hook label Jun 2, 2017
@efiop
Copy link
Copy Markdown
Contributor Author

efiop commented Jun 2, 2017

Hi @htgoebel ! I've added headers to v2. Please take a look.

Copy link
Copy Markdown
Member

@htgoebel htgoebel left a comment

Choose a reason for hiding this comment

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

Thanks. Strictly speaking, the data should be "2017" only, since the code of this hook was written this year, I assume. Would you mind to change this?

Otherwise this is okay.

@efiop
Copy link
Copy Markdown
Contributor Author

efiop commented Jun 2, 2017

Sure. Please see v3.

@htgoebel htgoebel merged commit ccf63e3 into pyinstaller:develop Jun 3, 2017
@htgoebel
Copy link
Copy Markdown
Member

htgoebel commented Jun 3, 2017

Thanks

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area:hooks Caused by or effecting some hook

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pyinstaller executable can't find package info

2 participants