Skip to content

Fix docgen script#990

Merged
taeold merged 8 commits intomasterfrom
dl-docgen-v1-fix
Oct 19, 2021
Merged

Fix docgen script#990
taeold merged 8 commits intomasterfrom
dl-docgen-v1-fix

Conversation

@taeold
Copy link
Copy Markdown
Contributor

@taeold taeold commented Oct 13, 2021

Notable changes:

  • We revert back prefixing v1 docs with v1_.
  • Dangling _ suffix is removed, e.g. cloud_functions_ => cloud_functions
  • Bunch more documents not in the ToC is created.
  • Some odd file naming, e.g. config.config-1.html instead of config.config1.html
  • Some type definiitions moved under common/ directory and hence the filename changed, e.g. providers_https is now common_providers_https.
  • Added more pages to provders.https namesapce (CallbleRequest, AppCheckData, AuthData).

@taeold taeold requested a review from egilmorez October 13, 2021 19:01
@google-cla google-cla bot added the cla: yes label Oct 13, 2021
Copy link
Copy Markdown
Contributor

@egilmorez egilmorez left a comment

Choose a reason for hiding this comment

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

Generated content looks good! Thanks for taking this on Daniel.

@taeold taeold merged commit 11e045a into master Oct 19, 2021
@taeold taeold mentioned this pull request Jan 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants