Skip to content

DOC Fix miniforge link with typo in install.rst#27019

Merged
betatim merged 1 commit intoscikit-learn:mainfrom
hiramatsuyuusuke:my_feature
Aug 7, 2023
Merged

DOC Fix miniforge link with typo in install.rst#27019
betatim merged 1 commit intoscikit-learn:mainfrom
hiramatsuyuusuke:my_feature

Conversation

@hiramatsuyuusuke
Copy link
Copy Markdown
Contributor

Reference Issues/PRs

What does this implement/fix? Explain your changes.

I fix miniforge link with typo in install.rst.
In line 64, from "https://https://github.com/conda-forge/miniforge#miniforge" to "https://github.com/conda-forge/miniforge#miniforge".
Additionally, some changes about trailing spaces are related to Shift_JIS with Windows.

Any other comments?

I am a GitHub beginner.
I read some guidelines, so I think this is correct PR.
If there are some faults about GitHub rules, I am sorry.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 5, 2023

✔️ Linting Passed

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

Generated for commit: bea65e5. Link to the linter CI: here

Copy link
Copy Markdown
Member

@betatim betatim left a comment

Choose a reason for hiding this comment

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

Thanks a lot for these improvements! They look good to me.

Welcome to the world of GitHub and contributing to open-source!

@betatim betatim merged commit 5f8d89e into scikit-learn:main Aug 7, 2023
@hiramatsuyuusuke hiramatsuyuusuke deleted the my_feature branch August 7, 2023 07:11
TamaraAtanasoska pushed a commit to TamaraAtanasoska/scikit-learn that referenced this pull request Aug 21, 2023
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Sep 18, 2023
REDVM pushed a commit to REDVM/scikit-learn that referenced this pull request Nov 16, 2023
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.

2 participants