Skip to content

[pull] master from jupyter-server:master #18

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

Open
wants to merge 287 commits into
base: master
Choose a base branch
from

Conversation

pull[bot]
Copy link

@pull pull bot commented Mar 9, 2021

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

@changelogg
Copy link

changelogg bot commented Mar 9, 2021

Hey! Changelogs info seems to be missing or might be in incorrect format.
Please use the below template in PR description to ensure Changelogg can detect your changes:
- (tag) changelog_text
or
- tag: changelog_text
OR
You can add tag in PR header or while doing a commit too
(tag) PR header
or
tag: PR header
Valid tags: added / feat, changed, deprecated, fixed / fix, removed, security, build, ci, chore, docs, perf, refactor, revert, style, test
Thanks!
For more info, check out changelogg docs

@codecov
Copy link

codecov bot commented Mar 9, 2021

Codecov Report

Merging #18 (6a3fc57) into master (36218db) will increase coverage by 0.13%.
The diff coverage is 76.66%.

❗ Current head 6a3fc57 differs from pull request most recent head 606487a. Consider uploading reports for the commit 606487a to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage   77.63%   77.76%   +0.13%     
==========================================
  Files         105      106       +1     
  Lines        8964     9269     +305     
  Branches      956      997      +41     
==========================================
+ Hits         6959     7208     +249     
- Misses       1666     1706      +40     
- Partials      339      355      +16     
Impacted Files Coverage Δ
jupyter_server/__main__.py 66.66% <ø> (-8.34%) ⬇️
jupyter_server/_sysinfo.py 81.25% <ø> (-0.57%) ⬇️
jupyter_server/files/handlers.py 29.16% <0.00%> (-1.95%) ⬇️
jupyter_server/nbconvert/handlers.py 22.77% <0.00%> (-0.23%) ⬇️
jupyter_server/services/contents/handlers.py 84.17% <ø> (ø)
jupyter_server/services/contents/manager.py 87.79% <0.00%> (-0.53%) ⬇️
jupyter_server/services/kernels/handlers.py 61.64% <0.00%> (+3.39%) ⬆️
jupyter_server/utils.py 62.50% <0.00%> (-0.97%) ⬇️
jupyter_server/base/handlers.py 63.76% <15.00%> (-4.48%) ⬇️
jupyter_server/terminal/handlers.py 76.00% <33.33%> (-8.22%) ⬇️
... and 23 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36218db...606487a. Read the comment docs.

@pull pull bot added the ⤵️ pull label Mar 9, 2021
blink1073 and others added 27 commits March 23, 2021 15:05
Ensure jupyter config dir exist
currently, all errors in pre_save_hook are ignored,
making it impossible for pre_save_hook to be used for validation / quota enforcement / etc.

By allowing HTTPErrors to raise, save will be cancelled and the error message will be shown to the user
allow pre_save_hook to cancel save with HTTPError
Update MappingKM.restart_kernel to accept now kwarg
Add descriptive log for port unavailable and port-retries=0
Before this PR, the default output of server urls looks like:

    Or copy and paste one of these URLs:
        http://localhost:8888/?token=…
     or http://127.0.0.1:8888/?token=…

This makes it easy to triple-click on the first line to copy and paste it into a browser. However, triple-clicking on the second line picks up the extra “or”, so you can’t just copy and paste it into the browser. Instead, you have to explicitly select each character by dragging your mouse, avoiding the “or”.

This change deletes the “or” so you can just triple-click on the second line to copy and paste the url.
Delete the extra “or” that prevents easy cut-and-paste of URLs.
Add support for JUPYTER_TOKEN_FILE
Add env variable support for port options
Update description for kernel restarted in the API docs
Remove unnecessary future imports
havok2063 and others added 28 commits November 2, 2021 14:08
* bug fixing allow_origin_pat property

* making requested changes
* Remove direct use of logging to keep log format consistent

* Add missing log for failed server info file deletion
* Add missing null check
* Remove unused import: logging
* enforce labels on PRs

* Update enforce-label.yml

* Update enforce-label.yml
avoids collisions across a given host
test_list_notebooks was failing on file systems with unicode
normalization different from the encoding used in the strings. In
particular on macOS with HFS+ I got failures for four of the file names.
This is because HFS+ uses normalization form D while normalization form
C is more commonly used, especially on Linux.

By normalizing the name and path read from disk to NFC we avoid the
mismatch as long as no NFD strings are added to the `dirs` tuples.
There are other unicode normal forms and normalization is not
necessarily invertible so this is not a perfect solution.

I can think of two alternative solutions:

1. Refrain from using any unicode symbols that have different
   representations in the various normal forms.
2. Write temporary files to disk somewhere, the read back the paths and
   filenames and use those instead of the original strings in the test.
* wip use pending kernels

yup

wip

wip

handle getattr

use ensure_future

add more backwards compat

clean up tests

more cleanup

Fix ping handling for shut down kernels

fix handling of kernel startup

lint

* fix handling of kernel activity

* clean up pre-commit

* make tests less brittle

* make tests less brittle

Co-authored-by: Steven Silvester <ssilvester@apple.com>
* Automated Changelog Entry for 1.12.0 on master

* Update CHANGELOG.md

Co-authored-by: GitHub Action <41898282+github-actions[bot]@users.noreply.github.com>
SHA256 hashes:

jupyter_server-1.12.0-py3-none-any.whl: b1f4a11a5a68f8aa4748e172e7b7789fa870f98b9b11f95f7530ea9ef581626c

jupyter_server-1.12.0.tar.gz: 0d8ab06f34e1cad224373e3b127e07d9cbb48babd66286136fbd075ce509b6ff
* await _finish_kernel_start

* Guard with use_pending_kernels
* Update to Python 3.10 in the CI workflows

* Lint
* Automated Changelog Entry for 1.12.1 on master

* Update CHANGELOG.md

Co-authored-by: GitHub Action <41898282+github-actions[bot]@users.noreply.github.com>
SHA256 hashes:

jupyter_server-1.12.1-py3-none-any.whl: 93a84d06c35613ecf3bc5de8ff2d92a410a3a5f57a3a23444ca75e4b2b390209

jupyter_server-1.12.1.tar.gz: f71e10ebaa6704a1e0fe76ec70a16a0804ab5a9d268f0c512e8c69086a8e86d1
* use tbump fields

* add version template

* fix name

* Update pyproject.toml
* clean up downstream tests

* syntax

* use new extra_test

* no unit tests

* fix option name
* allow the session manager to persist on disk

* validate the filepath given to a persistent session manager and add unit tests

* update help string

* finish comments in unit test

* precommit cleanup

* allow empty session datbase file
Co-authored-by: GitHub Action <41898282+github-actions[bot]@users.noreply.github.com>
SHA256 hashes:

jupyter_server-1.13.0-py3-none-any.whl: 608c0cb0cbbaa96a413c4d2427ca77acd14180251a2687831cf31f7648eaa835

jupyter_server-1.13.0.tar.gz: 05a96dfc5352adf70e88868b4ab3d1fc13e17bd84daf74d02cc6299098855fda
Co-authored-by: GitHub Action <41898282+github-actions[bot]@users.noreply.github.com>
SHA256 hashes:

jupyter_server-1.13.1-py3-none-any.whl: abfe55b6cd7bac0d7d7b8042765b7e451f11b5f2276a2ad708745cd8904d4e5b

jupyter_server-1.13.1.tar.gz: 6d70ebf8e789a7d0a5cd1588e078ccbbdca388dc2c74a6cd62b9ebb80609344f
significant time savings in pre-commit

pylint was only checking for one issue, which flake8 supports, but skipped

remove the skip in flake8 config, so pylint can be removed
@blink1073 blink1073 deleted the branch mtdev2:master December 13, 2021 12:22
@blink1073 blink1073 deleted the master branch December 13, 2021 12:22
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.