Skip to content

Updated doxygen options in docs.json #329

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

Closed

Conversation

SenRamakri
Copy link
Contributor

Sync-up doxygen_options.json changes from mbed-os repo to Handbook repo.

Copy link
Member

@pan- pan- left a comment

Choose a reason for hiding this comment

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

That patch would prevent BLE documentation to be generated by doxygen. I don't think it is expected.

@AnotherButler
Copy link
Contributor

I believe features/filesystem has been taken off the excluded list.

@SenRamakri
Copy link
Contributor Author

@pan- I'm trying to sync-up the doxygen_options.json from Handbook repo. If you think the changes are not good for BLE documentation we have to update that in Handbook repo as well. What paths/targets are missing?

@pan-
Copy link
Member

pan- commented Nov 7, 2017

@SenRamakri 99818b2 removed FEATURE_ from docs.json the 25th of september; this change was not reflected to doxyfile_options.json in mbed repository.

@AnotherButler Could you indicate if 99818b2 should be reflected in mbed repository ?

@AnotherButler
Copy link
Contributor

The two should be the same, and the handbook should match the mbed-os one, not the other way around. I thought @kegilbert put in a PR to mbed-os so that features/filesystem (not FEATURES) would no longer be excluded.

@kegilbert
Copy link
Contributor

ARMmbed/mbed-os#5395

Should cover it, not sure if we need a manual rebuild of docs to pull in those changes.

@AnotherButler
Copy link
Contributor

I believe this has been superseded by #332

@SenRamakri Is it OK to close this in favor of 332?

@SenRamakri
Copy link
Contributor Author

@AnotherButler , I think yes, But can you please add the PROJECT_NAME as below.
"PROJECT_NAME": "Mbed OS Reference",

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants