Skip to content

Migrate to src layout#719

Merged
fderuiter merged 2 commits intomainfrom
refactor/src-layout-migration-5707453264421547486
Feb 27, 2026
Merged

Migrate to src layout#719
fderuiter merged 2 commits intomainfrom
refactor/src-layout-migration-5707453264421547486

Conversation

@fderuiter
Copy link
Owner

Migrated the project to use the standard src layout. This involves moving the source code into a src/ directory, which helps prevent import errors during testing (testing against the installed package rather than the local folder) and is a recommended practice for modern Python packaging.

Changes:

  • created src/ directory.
  • moved imednet/ -> src/imednet/.
  • updated pyproject.toml packages config.
  • updated Makefile for sphinx-apidoc paths.
  • updated docs/conf.py sys.path to include ../src.
  • updated CI/CD workflow to type check src/imednet.

PR created automatically by Jules for task 5707453264421547486 started by @fderuiter

Moved the `imednet` package to `src/imednet` to adopt the standard Python src layout. This change improves packaging isolation and testing practices.

- Moved `imednet/` to `src/imednet/`.
- Updated `pyproject.toml` to point to the new package location (`packages = [{ include = "imednet", from = "src" }]`).
- Updated `Makefile` and `docs/conf.py` to reference the new source path for documentation generation.
- Updated `.github/workflows/main.yml` to run mypy against `src/imednet`.
- Verified installation, testing, and documentation generation.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
@google-labs-jules
Copy link
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

Ran `isort` on the codebase to fix import sorting errors in `examples/airflow/` that were flagged by the CI pipeline. This aligns the examples with the updated `src` layout configuration where `imednet` is recognized as a first-party package.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
@fderuiter fderuiter marked this pull request as ready for review February 27, 2026 18:03
@fderuiter fderuiter merged commit 45ac097 into main Feb 27, 2026
13 checks passed
@fderuiter fderuiter deleted the refactor/src-layout-migration-5707453264421547486 branch February 27, 2026 18:03
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.

1 participant