Skip to content

Conversation

@pdtxie
Copy link
Contributor

@pdtxie pdtxie commented Apr 17, 2025

mirror of #623

@adnanhemani
Copy link
Member

Seems like we have bigger failure issues with the repository build :( Will look into these hopefully soon and then get back to this - thank you for opening this!

@adnanhemani
Copy link
Member

@pdtxie - I think we've fixed the build. Can you put some small change to re-trigger the CI/CD?

@pdtxie
Copy link
Contributor Author

pdtxie commented May 25, 2025

Just added an example in the docstring

@coveralls
Copy link

Coverage Status

coverage: 94.976%. remained the same
when pulling 6369ce2 on pdtxie:master
into cf403b5 on data-8:master.

Copy link
Member

@adnanhemani adnanhemani left a comment

Choose a reason for hiding this comment

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

LGTM - thank you for contributing this!

@adnanhemani adnanhemani merged commit 0385222 into data-8:master May 27, 2025
1 check 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.

3 participants