Skip to content

Conversation

@pjcollins
Copy link
Member

The mdoc export-msxdoc command that runs at the end of the API docs
build shouldn't overwrite the Mono.Android.xml file produced by the
build.

The `mdoc export-msxdoc` command that runs at the end of the API docs
build shouldn't overwrite the `Mono.Android.xml` file produced by the
build.
Copy link
Member

@jonathanpeppers jonathanpeppers left a comment

Choose a reason for hiding this comment

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

This looks OK to me. What problem was caused by this, can we put that in the final commit message? It seems like something exporting docs was probably broken.

@pjcollins
Copy link
Member Author

We want to run this step during the regular build, but during the docs build we want to preserve the XML docs that have the additional content. I'll add a note

@pjcollins pjcollins merged commit adca983 into main Oct 24, 2023
@pjcollins pjcollins deleted the dev/pjc/optional-xml-export branch October 24, 2023 15:49
grendello added a commit that referenced this pull request Oct 24, 2023
* main:
  [ci] Stop building monodroid in builds from forks (#8444)
  [docs] Don't overwrite xml produced by docs build (#8451)
  Bump NDK to r26b (#8450)
  [Tests] Fix TestResolveToolsExists on .NET (#8445)
@github-actions github-actions bot locked and limited conversation to collaborators Jan 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants