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.

Sign up for GitHub

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

Jump to bottom

Write MLJ-compliant docstrings #32

Closed
ablaom opened this issue Jun 29, 2023 · 0 comments · Fixed by #37
Closed

Write MLJ-compliant docstrings #32

ablaom opened this issue Jun 29, 2023 · 0 comments · Fixed by #37

Comments

@ablaom
Copy link

ablaom commented Jun 29, 2023

Be good to review the current doc-strings to conform with the official spec..

Also the docstring trait, as declared here is deprecated and should be removed. Are you not already getting warnings about this?
@ablaom ablaom changed the title Write MLJ compliant docstrings Write MLJ-compliant docstrings Jun 29, 2023
@ablaom ablaom mentioned this issue Jun 30, 2023
Closed
@rikhuijzer rikhuijzer mentioned this issue Jul 9, 2023
Merged
rikhuijzer added a commit that referenced this issue Jul 9, 2023
@rikhuijzer
Write MLJ-compliant docstrings (#37)
16d4c59 
Fixes #32.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Write MLJ-compliant docstrings rikhuijzer/SIRUS.jl
1 participant
@ablaom