Skip to content

Fix cross references in generated API references #48

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 1 commit into from
May 31, 2018

Conversation

tengqm
Copy link
Contributor

@tengqm tengqm commented Apr 12, 2018

In previous commit, we have enabled the API reference doc to show full
group names, however, due to implementation constraints (the use of
brodocs docker image), the reformatted H1 headers are not producing
anchor IDs properly. This PR reverses the H1 header generated.
Other issues fixed:

  • .gitignore pointing to some non-existent files/dirs
  • config.yaml was using v1alpha1 version of VolumeAttachment as default,
    so the navigation bar is not working properly.
  • config.yaml was using v1beta1 version of APIService which prevents the
    navigation bar from working properly.
  • Added handler to special character sequences such as '', '/scale',
    'pods/' and '/*' which had been incorrectly translated into
    by the markdown Node.js module.
  • Other links fixed for cross-references.

Will continue digging how to simplify the reference doc generation.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 12, 2018
@tengqm
Copy link
Contributor Author

tengqm commented Apr 25, 2018

/retest

@tengqm
Copy link
Contributor Author

tengqm commented Apr 25, 2018

Have to close and reopen to trigger a retest?

@tengqm tengqm closed this Apr 25, 2018
@tengqm tengqm reopened this Apr 25, 2018
@steveperry-53
Copy link
Contributor

/approve
@tengqm Can you fix the merge conflict. Then I'll merge this PR.

In previous commit, we have enabled the API reference doc to show full
group names, however, due to implementation constraints (the use of
brodocs docker image), the reformatted H1 headers are not producing
anchor IDs properly. This PR reverses the H1 header generated.
Other issues fixed:

- .gitignore pointing to some non-existent files/dirs
- config.yaml was using v1alpha1 version of VolumeAttachment as default,
  so the navigation bar is not working properly.
- config.yaml was using v1beta1 version of APIService which prevents the
  navigation bar from working properly.
- Added handler to special character sequences such as '*', '*/scale',
  'pods/*' and '*/*' which had been incorrectly translated into <em>
  by the markdown Node.js module.
- Other links fixed for cross-references.

Will continue digging how to simplify the reference doc generation.
@tengqm tengqm force-pushed the fix-apiref-links branch from 44c29c5 to 13682fc Compare May 31, 2018 02:28
@tengqm
Copy link
Contributor Author

tengqm commented May 31, 2018

@steveperry-53 Done.

@steveperry-53 steveperry-53 merged commit 7b1f589 into kubernetes-sigs:master May 31, 2018
@tengqm tengqm deleted the fix-apiref-links branch July 18, 2018 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants