Skip to content

Conversation

@maharshi-gor
Copy link
Member

No description provided.

@maharshi-gor maharshi-gor requested a review from skoudoro January 5, 2026 16:38
@maharshi-gor maharshi-gor self-assigned this Jan 5, 2026
@github-actions
Copy link

github-actions bot commented Jan 5, 2026

🪓 PR closed, deleted preview at https://github.com/dipy/preview-html/tree/main/workshop.dipy.org/pull/47/

Copy link
Member

@skoudoro skoudoro left a comment

Choose a reason for hiding this comment

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

Hi @maharshi-gor,

Thank you, here a quick comment to check

- id: codespell
additional_dependencies:
- tomli
default_language_version:
Copy link
Member

Choose a reason for hiding this comment

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

What are the update on this file? Can you make sure to avoid adding extra windows character ? maybe we should add this check on pre-commit

@maharshi-gor
Copy link
Member Author

Thanks for pointing out @skoudoro I have fixed it manually but let me see what can I do to have LF style instead of CRLF on command line generated stuff. Also, let's block it from the pre-commit check so it will not have the same issue.

@skoudoro skoudoro merged commit ba4281f into dipy:main Jan 6, 2026
3 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