Skip to content

Conversation

@divakaivan
Copy link
Contributor

Reference Issues/PRs

None

What does this implement/fix? Explain your changes.

  • Remove duplicated sentence
  • Fix typos/punctuation.

Copy link
Member

@adrinjalali adrinjalali left a comment

Choose a reason for hiding this comment

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

Thanks @divakaivan

@divakaivan
Copy link
Contributor Author

divakaivan commented Oct 2, 2025

Hmm... the pytest (windows-lated, ci-sklearn16) seems unrelated, but I reran the github action on me fork and it passes 🤔

@adrinjalali
Copy link
Member

That one was a fluke, the other ones are known, we need to ignore the warning coming from PassiveAggressiveRegressor (and I'd welcome a PR doing that 😉 )

@adrinjalali adrinjalali merged commit 0fa536e into skops-dev:main Oct 3, 2025
36 of 43 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.

2 participants