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

Iris API param not rendering #4657

Closed
tkknight opened this issue Mar 24, 2022 · 5 comments
Closed

Iris API param not rendering #4657

tkknight opened this issue Mar 24, 2022 · 5 comments
Assignees

Comments

@tkknight
Copy link
Contributor

tkknight commented Mar 24, 2022

📚 Documentation

Whilst reviewing the dsocs for another task I noticed there are a few occurrences of :param: that is not rendering correctly due to alignment and maybe other syntax issues.

There maybe other occurrences too, not dug any further.

@trexfeathers
Copy link
Contributor

I didn't even know that was supposed to be possible! Would be great if it worked 🙂

@pp-mo
Copy link
Member

pp-mo commented Mar 24, 2022

It definitely does work in many places.
For example : https://scitools-iris.readthedocs.io/en/latest/generated/api/iris/coords.html#iris.coords.CellMethod.method
which comes from : https://github.com/SciTools/iris/blob/main/lib/iris/coords.py#L2974-L2975

However, I'm seeing that some other things are not so happy..
Sphinx seems to have started putting two of these on one line sometimes. #4660
Which may or not be related..

@vsherratt
Copy link
Contributor

The consistent thing between those examples is that they are formatted like

Args:
    * something - description

which is probably a few too many steps away from what napoleon expects. I would try:

  • removing the * list markers
  • removing the indentation
  • replacing the - with :

@tkknight
Copy link
Contributor Author

I have a branch primed to fix this issue but ideally want to use my read the docs test space to render them for all to see, thish is being used for another PR atm. Will create the PR next week sometime.

Related to #3840.

@tkknight
Copy link
Contributor Author

Fixed by #3840.

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

No branches or pull requests

4 participants