Skip to content

Conversation

@graingert
Copy link
Member

@graingert graingert commented Feb 16, 2022

this uses "importlib.metadata:version" - for version which is much faster than git

@github-actions
Copy link
Contributor

github-actions bot commented Feb 17, 2022

Unit Test Results

       11 files  ±  0         11 suites  ±0   6h 8m 4s ⏱️ - 24m 53s
  2 620 tests +  2    2 539 ✔️ +  4    81 💤 ±0  0  - 2 
14 443 runs  +12  13 605 ✔️ +14  838 💤 ±0  0  - 2 

Results for commit 8947abd. ± Comparison against base commit 9a266a0.

♻️ This comment has been updated with latest results.

@graingert graingert force-pushed the faster-dunder-version branch from 02941db to 7d445d4 Compare February 17, 2022 10:04
@graingert graingert marked this pull request as ready for review February 17, 2022 10:09
@graingert graingert requested a review from fjetter February 17, 2022 17:06
@graingert graingert force-pushed the faster-dunder-version branch from 7d445d4 to b1c6b0d Compare February 18, 2022 08:26
@fjetter
Copy link
Member

fjetter commented Feb 18, 2022

Your file formatting is a bit off. I would suggest to install pre-commit, see http://distributed.dask.org/en/stable/develop.html#code-formatting

@graingert graingert force-pushed the faster-dunder-version branch from b1c6b0d to 3578e4c Compare February 22, 2022 09:54
this uses "importlib.metadata:version" - for __version__ which is much faster than git
@graingert graingert force-pushed the faster-dunder-version branch from 3578e4c to 8947abd Compare February 22, 2022 15:47
@graingert graingert requested a review from fjetter February 22, 2022 16:22
@fjetter fjetter merged commit 158a3e7 into dask:main Feb 23, 2022
@graingert graingert deleted the faster-dunder-version branch February 23, 2022 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

importing distributed runs 4 git subprocesses in CI (when installed with -e)

2 participants