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

Extension Compatiblity #2410

Merged
merged 20 commits into from
Jun 15, 2017
Merged

Conversation

blink1073
Copy link
Contributor

@blink1073 blink1073 commented Jun 8, 2017

Fixes #2356.

Note: Populated the singletonPackages using the packages that export a token and @phosphor/widgets, which comprises the packages that are explicitly shared.

Sample output:

ValueError:
"@jupyterlab/python-tests@0.1.0" is not compatible with the current JupyterLab
Conflicting Dependencies:
Required	Actual	Package
^0.6.0  	^0.5.0	@jupyterlab/application

TODO:

  • Update the release script to add the explicit deps from the listed extensions
  • Use those deps along with singleton packages config to determine compat
  • Use the compat information in list - show as invalid, offer verbose mode with full report
  • Use the compat information when building - remove from templated package.json
  • Add tests

@blink1073
Copy link
Contributor Author

Going to wait until #2409 is merged to finish this, since there will be a lot of overlap.

@blink1073 blink1073 changed the title [WIP] Extension Compatiblity Extension Compatiblity Jun 14, 2017
@blink1073 blink1073 mentioned this pull request Jun 14, 2017
@ellisonbg
Copy link
Contributor

Trying some things out with this branch:

First I ran:

jupyter labextension uninstall @jupyterlab/help-extension

I then saw this:

(python34) $ jupyter labextension list
[ListLabExtensionsApp] JupyterLab v0.23.2
[ListLabExtensionsApp] Known labextensions:
[ListLabExtensionsApp] 
Uninstalled core extensiosn:
[ListLabExtensionsApp]     @jupyterlab/help-extension

But when I tried to install that extension again I got a Python exception that ended with:

ValueError: @jupyterlab/help-extension is not a valid JupyterLab extension

@ellisonbg
Copy link
Contributor

I tried to link/unlink the mockextension and mockextension-incompat and it all worked as expected.


# Handle uninstalled and disabled core packages
uninstalled_core = _get_uinstalled_core_extensions(app_dir)
if uninstalled_core:
print('\nUninstalled core extensiosn:')
[print(' %s' % item) for item in sorted(uninstalled_core)]
logger.info('\nUninstalled core extensiosn:')
Copy link
Contributor

Choose a reason for hiding this comment

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

extensiosn

@ellisonbg
Copy link
Contributor

Tested latest changes locally and all is happy. Once the tests pass, let's merge!

@ellisonbg
Copy link
Contributor

Come on Travis!

@ellisonbg ellisonbg merged commit cc55794 into jupyterlab:master Jun 15, 2017
@blink1073 blink1073 deleted the extension-compat branch July 10, 2017 17:56
@lock lock bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Aug 10, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Aug 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. tag:Build System tag:Extensions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extension compatibility check
2 participants