Skip to content

CDRIVER-5565 remove non-API docs #1600

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 31 commits into from
May 16, 2024
Merged

CDRIVER-5565 remove non-API docs #1600

merged 31 commits into from
May 16, 2024

Conversation

kevinAlbs
Copy link
Collaborator

@kevinAlbs kevinAlbs commented May 15, 2024

Summary

  • Remove documentation migrated to MongoDB C Driver and update links.
  • Refer to new documentation from landing pages.

Rendered pages for the libmongoc and libbson projects are viewable from the Files tab of the make-docs Evergreen task.

Background & Motivation

Documentation has been migrated as part of DOCSP-29036. The DBX C/C++ team is expected to continue owning the API documentation.

CONTRIBUTING.md describes building docs with poetry. I use sphinx-autobuild, which starts a local server and auto-reloads:

# Build libmongoc docs:
DIR=src/libmongoc/doc
OUTDIR=cmake-build/src/libmongoc/doc/html
poetry run sphinx-autobuild -b html $DIR $OUTDIR --re-ignore ".*.pickle" --re-ignore ".*.doctree" -j auto --port 9000

kevinAlbs added 28 commits May 15, 2024 14:04
The API and APM docs are linked to from mongodb.com.
The index page only shows one entry for `mongoc_kms_credentials_provider_callback_fn`.
The API docs are linked to from mongodb.com.
@kevinAlbs kevinAlbs marked this pull request as ready for review May 15, 2024 19:30
@kevinAlbs kevinAlbs requested a review from rcsanchez97 May 15, 2024 19:32
@kevinAlbs kevinAlbs requested a review from kkloberdanz May 15, 2024 19:47
Copy link
Contributor

@kkloberdanz kkloberdanz left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@rcsanchez97 rcsanchez97 left a comment

Choose a reason for hiding this comment

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

LGTM, with some recommended tweaks (at your discretion).

@kevinAlbs
Copy link
Collaborator Author

Merging. The debian-package-build task failed. Re-running the task on the base commit resulted in the same error:

mv: cannot stat '/data/mci/efd2c84b56670324bb69fc93a4be1e11/unstable-chroot/sbin/*': No such file or directory

I expect it is unrelated to this PR.

@kevinAlbs kevinAlbs merged commit 151975f into mongodb:master May 16, 2024
40 of 42 checks passed
kevinAlbs added a commit that referenced this pull request May 16, 2024
* libmongoc: remove links to content moved to mongodb.com

The API and APM docs are linked to from mongodb.com.

* libmongoc: remove no-longer-referenced files

* libmongoc: remove `genindex`

The index page only shows one entry for `mongoc_kms_credentials_provider_callback_fn`.

* libmongoc: link to `MongoDB C Driver` page

* libmongoc: fix links to `queryable-encryption`

* libmongoc: fix links to `authentication`

* libmongoc: fix links to `bulk`

* libmongoc: fix links to `client-side-field-level-encryption`

* libmongoc: fix links to `configuring_tls`

* libmongoc: fix links to `connection-pooling`

* libmongoc: fix links to `in-use-encryption`

* libmongoc: fix links to `manage-collection-indexes`

* libmongoc: fix links to ``setting_collation_order`

* libmongoc: fix links to `tutorial_crud_operations`

* libmongoc: fix link to `bulk_operation_bypassing_document_validation`

* libmongoc: fix links to `cursors_tailable`

* libbson: remove links to content moved to mongodb.com

The API docs are linked to from mongodb.com.

* libbson: link to `MongoDB C Driver` page

* libbson: remove no-longer-referenced files

* libbson: fix links to `errors`

* libmongoc: fix links to `bson:errors`

* libmongoc: fix links to `bson:lifetimes`

* homepage: link to `MongoDB C Driver` page

* libmongoc: remove no-longer-used images

* libmongoc: remove unreferenced include

* update links in README

* update link in CONTRIBUTING.md

* fix links to community forums

* add missing "the"

Co-authored-by: Roberto C. Sánchez <[email protected]>

* fix `setting_collation_order_` links

Co-authored-by: Roberto C. Sánchez <[email protected]>

* fix `setting_collation_order_` in `generate-opts.py`

---------

Co-authored-by: Roberto C. Sánchez <[email protected]>
@rcsanchez97
Copy link
Contributor

Yeah, that failure looks completely unrelated. I will take a look at the task status on the waterfall and confirm.

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.

3 participants