-
Notifications
You must be signed in to change notification settings - Fork 4
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
Added lightning package to setup file and updated version for re… #69
Conversation
… blacklist/exclude package import style in setup.py
@divrawal can you update the PR title/description to describe what you're trying to accomplish with this PR? |
Approving this in light of the follow-up task b/356629540. Divya is adding all the details to that ticket. In short we will let this one pass to unblock the other work. But we should prioritize b/356629540 because similar changes here are risky -- we should rely on |
@divrawal is this ready to merge? |
* Added lightning subdirectory to setup file and updated version for release * Modified packages to instead exclude the testing packages * Exclude top level package * Added back in package_dir field to setup.py * Added back in package_dir field to setup.py * Attempting import fix * Reverted it to working, whitelist build- Will open a bug to address a blacklist/exclude package import style in setup.py * Typo
new release with updated lightning dependency