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

Revert "Fix some CodeQL notices" #372

Merged
merged 1 commit into from
Feb 7, 2023
Merged

Conversation

fcollonval
Copy link
Member

Reverts #371 partly to restore server.py that may be consumed by downstream package

@fcollonval fcollonval marked this pull request as ready for review February 7, 2023 09:43
@fcollonval fcollonval enabled auto-merge (squash) February 7, 2023 09:46
Comment on lines +8 to +13
from jupyter_server.base.handlers import (
APIHandler,
FileFindHandler,
JupyterHandler,
json_errors,
)

Check notice

Code scanning / CodeQL

Unused import

Import of 'APIHandler' is not used. Import of 'FileFindHandler' is not used. Import of 'JupyterHandler' is not used. Import of 'json_errors' is not used.
Comment on lines +14 to +19
from jupyter_server.extension.serverextension import (
GREEN_ENABLED,
GREEN_OK,
RED_DISABLED,
RED_X,
)

Check notice

Code scanning / CodeQL

Unused import

Import of 'GREEN_ENABLED' is not used. Import of 'GREEN_OK' is not used. Import of 'RED_DISABLED' is not used. Import of 'RED_X' is not used.
"""Server api."""
# FIXME TODO Deprecated remove this file for the next major version
# Downstream package must import those items from `jupyter_server` directly
from jupyter_server import _tz as tz

Check notice

Code scanning / CodeQL

Unused import

Import of 'tz' is not used.
RED_DISABLED,
RED_X,
)
from jupyter_server.serverapp import ServerApp, aliases, flags

Check notice

Code scanning / CodeQL

Unused import

Import of 'ServerApp' is not used. Import of 'aliases' is not used. Import of 'flags' is not used.
RED_X,
)
from jupyter_server.serverapp import ServerApp, aliases, flags
from jupyter_server.utils import url_escape, url_path_join

Check notice

Code scanning / CodeQL

Unused import

Import of 'url_escape' is not used. Import of 'url_path_join' is not used.
@fcollonval fcollonval merged commit eb37a21 into main Feb 7, 2023
@fcollonval fcollonval deleted the revert-371-maintenance/codeql branch February 7, 2023 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant