Switch Python package to src-layout #865
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ml-evs
force-pushed
the
ml-evs/fix-python-package
branch
3 times, most recently
from
aa50729
to
f4d04ab
Compare
ml-evs
force-pushed
the
ml-evs/fix-python-package
branch
from
16a1384
to
0e15744
Compare
ml-evs
force-pushed
the
ml-evs/fix-python-package
branch
from
0e15744
to
2a2f45e
Compare
datalab
|
||||||||||||||||||||||||
| Project |
datalab
|
| Run status |
|
| Run duration | 06m 22s |
| Commit |
|
| Committer | Matthew Evans |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
Failures
|
0
|
Flaky
|
0
|
Pending
|
0
|
Skipped
|
0
|
Passing
|
396
|
ml-evs
force-pushed
the
ml-evs/fix-python-package
branch
3 times, most recently
from
52887ea
to
897e715
Compare
ml-evs
force-pushed
the
ml-evs/fix-python-package
branch
from
897e715
to
249336d
Compare
ml-evs
added
build
ml-evs
force-pushed
the
ml-evs/fix-python-package
branch
from
249336d
to
01386ea
Compare
BenjaminCharmes
previously approved these changes
Oct 7, 2024
Remove defunct version setter Update configs for src-layout, and move server deps to default Fix mkdocs config Fix pre-commit config Fix links in pyproject Fix links in pyproject Add explicit known-first-party to avoid reformatting tests/scripts Move core server deps to defaults Tweak pyproject for src-layout `pre-commit autoupdate`, minus eslint
Remove now-unecessary docker experimental syntax pragma Matching CI ruff results; not sure why I cannot reproduce locally
Fix syntax warning in dockerfile Update dockerfile for src-layout Rejig dev api container to accommodate src-layout and add dev container healthcheck Fix docker build Disable fail fast Combine dev docker builds into one job
ml-evs
force-pushed
the
ml-evs/fix-python-package
branch
from
fac2f36
to
035c1da
Compare
This was referenced Oct 8, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Switching to
src-layout is increasingly helpful for plugin development and avoids misleading imports. The main knock-on effects are related to deployment, as well as some minor tweaks needed to launch the Python server.