revert to synchronous contents manager, add separate async contents manager

[timkpaine]

No description provided.

[github-actions]

Launch a binder notebook on branch timkpaine/jupyter-fs/tkp/cm

[codecov]

Codecov Report

Attention: Patch coverage is 87.32394% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 35.26%. Comparing base (1a84b01) to head (c0748f5).

❗ Current head c0748f5 differs from pull request most recent head fc4d3d4

Please upload reports for the commit fc4d3d4 to get more accurate results.

Files	Patch %	Lines
jupyterfs/pathutils.py	77.41%	4 Missing and 3 partials ⚠️
jupyterfs/extension.py	50.00%	0 Missing and 1 partial ⚠️
jupyterfs/metamanager.py	96.87%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #186      +/-   ##
==========================================
+ Coverage   34.83%   35.26%   +0.43%     
==========================================
  Files          37       37              
  Lines        2589     2651      +62     
  Branches      458      465       +7     
==========================================
+ Hits          902      935      +33     
- Misses       1543     1569      +26     
- Partials      144      147       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[vidartf]

Shouldn't these be sync=True for the SynceMetaManager version to be correct?

[vidartf]

Sorry for the delay in getting to this. I think the code looks good other than the sync=True/False being flipped, so once that is fixed I think we can merge and release.

[timkpaine]

Let me check, the tests should definitely fail if this was done wrong