G
YouTube TikTok Games
Skip to content

docs: remove stale src/docpipe_app/backend references - #85

Merged
Adityars-ibm merged 1 commit into
docling-project:mainfrom
reginaldalfret:fix/19-stale-doc-paths
Oct 5, 2026
Merged

Adityars-ibm merged 1 commit into
docling-project:mainfrom
reginaldalfret:fix/19-stale-doc-paths

Conversation

@reginaldalfret

Copy link
Copy Markdown
Contributor

Summary

  • Updated contributor documentation to use the current project root layout and virtual environment activation path.
  • In docs/guides/CREATE_CONNECTOR_GUIDE.md, replaced obsolete source src/docpipe_app/backend/.venv/bin/activate paths with source .venv/bin/activate.
  • In docs/internals/DOCUMENT_LIBRARIES_ARCHITECTURE.md, replaced obsolete backend directory navigation and nested PYTHONPATH with root project layout (export PYTHONPATH="$(pwd)/src:${PYTHONPATH}"), and simplified pytest paths accordingly.
  • Removed all obsolete src/docpipe_app/backend references from the documentation tree.

Testing

  • Validated repository-wide documentation search:
    git grep -ni "docpipe_app" docs/ -> 0 matches.
  • Validated git diff --check -> clean (no whitespace or formatting issues).
  • Verified paths match current project layout: .venv/bin/activate at project root and $(pwd)/src containing the docpipe package.

Fixes #19

@github-actions

Copy link
Copy Markdown

✅ DCO Check Passed

Thanks @reginaldalfret, all your commits are properly signed off. 🎉

@Adityars-ibm

Copy link
Copy Markdown
Collaborator

Hi @reginaldalfret
Please follow https://docs.github.com/en/authentication/managing-commit-signature-verification to add verified commits. Thanks

Update contributor guides and architecture documentation to use the current project root layout and virtual environment activation path.

- In docs/guides/CREATE_CONNECTOR_GUIDE.md, update venv activation path to .venv/bin/activate.

- In docs/internals/DOCUMENT_LIBRARIES_ARCHITECTURE.md, replace backend directory navigation and nested PYTHONPATH with root project layout.

Fixes docling-project#19

Signed-off-by: reginaldalfret <reginaldalfret@gmail.com>

@jo-in jo-in left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mergify

mergify Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@reginaldalfret

Copy link
Copy Markdown
Contributor Author

@Mergifyio queue

@mergify

mergify Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

queue

☑️ Command disallowed due to command restrictions in the Mergify configuration.

Details
  • sender-permission >= write

@Adityars-ibm
Adityars-ibm merged commit 9423533 into docling-project:main Oct 5, 2026
5 checks passed
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.

docs: remove stale src/docpipe_app/backend references from contributor guides

3 participants