Skip to content

Conversation

charris
Copy link
Member

@charris charris commented Feb 14, 2025

This makes the c/c++ line length the same as the modern Python defaults. Note that linux has dropped the hard 80 character limit, it is now only preferred. I don't know how to configure a linter for "preferred", so just make the hard limit 88.

This makes the c/c++ line length the same as the modern Python defaults.
Note that linux has dropped the hard 80 character limit, it is now only
preferred. I don't know how to configure a linter for "preferred", so
just make the hard limit 88.
@r-devulap
Copy link
Member

Do we want to run this on the repository to reformat the existing code base?

@mattip
Copy link
Member

mattip commented Feb 18, 2025

I think reformatting all the code would create too much noise for the benefit gained. But if there are particular areas of the code that could be helped, we should reformat them selectively.

@mattip mattip merged commit e327686 into numpy:main Feb 18, 2025
68 of 69 checks passed
@mattip
Copy link
Member

mattip commented Feb 18, 2025

Thanks @charris

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.

3 participants