Skip to content
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

Bump docfx version #88

Merged
merged 1 commit into from May 13, 2024
Merged

Conversation

sanych-sun
Copy link
Member

@sanych-sun sanych-sun commented May 2, 2024

Bumped DOCFX version to support the latest language features (primary constructors first of all)
Added API docs validation script to be runned on Main and PR build for earlier problems detection.

@sanych-sun sanych-sun requested a review from a team as a code owner May 2, 2024 23:25
@@ -14,7 +14,8 @@
"disableDefaultFilter": false,
"noRestore": false,
"namespaceLayout": "flattened",
"memberLayout": "samePage",
"memberLayout": "separatePages",
Copy link
Member Author

Choose a reason for hiding this comment

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

Needed for easier access to specific member pages.

@@ -224,6 +224,7 @@ functions:

pack-packages:
- command: shell.exec
type: test
Copy link
Member Author

Choose a reason for hiding this comment

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

This will mark failed script as "Failed" (red) instead of "System Failure" (purple)

params:
shell: bash
working_dir: mongo-efcore-provider
script: |
${PREPARE_SHELL}
if ! [[ "$PACKAGE_VERSION" =~ ^[0-9]+\.[0-9]+\.0$ ]]; then
Copy link
Member Author

Choose a reason for hiding this comment

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

Irrelevant, as we run build-apidocs for each commit now.

@@ -502,22 +513,24 @@ buildvariants:
tasks:
- name: packages-tests
git_tag_only: true
priority: 10
Copy link
Member Author

Choose a reason for hiding this comment

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

Set priority for release-related tasks to prioritize package creation over regular tasks.

@sanych-sun sanych-sun requested review from damieng and removed request for a team May 3, 2024 00:26
Copy link
Member

@damieng damieng left a comment

Choose a reason for hiding this comment

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

LGTM,

Thanks for sorting this out!

@sanych-sun sanych-sun changed the title [WIP] Bump docfx version Bump docfx version May 3, 2024
@sanych-sun sanych-sun merged commit c40a672 into mongodb:main May 13, 2024
34 checks passed
@sanych-sun sanych-sun deleted the bump-docfx-version branch May 13, 2024 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants