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

DOC Fix gramatical error in governence #30210

Merged
merged 3 commits into from
Nov 5, 2024

Conversation

rajathkannabiran
Copy link
Contributor

@rajathkannabiran rajathkannabiran commented Nov 4, 2024

Reference Issues/PRs

Fixes #30180 and #30181

What does this implement/fix? Explain your changes.

Fixing a simple grammatical mistake in https://scikit-learn.org/stable/governance.html
as per Issue #30180 by @AdityaInnovates

Copy link

github-actions bot commented Nov 4, 2024

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 84a1621. Link to the linter CI: here

Comment on lines 160 to 162
* **Changes to the API principles and changes to dependencies or supported
versions** happen via a :ref:`slep` and follows the decision-making process
versions** happen via an :ref:`slep` and follows the decision-making process
outlined above.
Copy link
Member

Choose a reason for hiding this comment

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

Instead, you can do

* **Changes to the API principles and changes to dependencies or supported versions**
  happen via :ref:`Enhacement Proposals (SLEPs) <slep>` and follows the
  decision-making process outlined above.

@adrinjalali adrinjalali changed the title Issue: 30180 Fix gramatical error DOC Fix gramatical error in governence Nov 4, 2024
@adrinjalali
Copy link
Member

Please apply the issues from #30181 in the same PR.

@rajathkannabiran
Copy link
Contributor Author

@adrinjalali made the recommended change and included #30181 too in this PR. Please review

@adrinjalali adrinjalali merged commit c634c49 into scikit-learn:main Nov 5, 2024
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC grammar issue in the governance page
2 participants