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

make full use of autodoc such that cross referencing can find all targets #54

Merged
merged 7 commits into from
Mar 5, 2024

Conversation

lukashergt
Copy link
Collaborator

This PR is motivated by a docs related issue that came up in handley-lab/anesthetic#353.

Currently the docs are generated in a way that does not allow cross-referencing of all modules and its members. This PR should (hopefully) remedy that. This comes with the added benefit of simplifying the docs source code, leveraging more of the autodoc power.

@htjb
Copy link
Owner

htjb commented Dec 14, 2023

Nice this is very cool thanks @lukashergt!

Should wait for #53 before merging this in.

@ThomasGesseyJones
Copy link
Collaborator

Just to let you know, PR #53 has now been merged

@htjb
Copy link
Owner

htjb commented Dec 18, 2023

Great looks good thanks again @lukashergt! I sent you a collaborator request so you can squash and merge 🚀

@htjb
Copy link
Owner

htjb commented Feb 9, 2024

Hi @lukashergt I think might need to pull master branch back in but otherwise happy for this to be merged! Thanks again.

@lukashergt
Copy link
Collaborator Author

Hi @htjb, sorry I missed the invitation, and it now is expired.

@htjb
Copy link
Owner

htjb commented Mar 5, 2024

@lukashergt No problem! Just re sent it.

@lukashergt lukashergt merged commit d764234 into htjb:master Mar 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants