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

Extend black formatting to look in the test directory too #137

Open
liamhuber opened this issue Dec 17, 2024 · 2 comments
Open

Extend black formatting to look in the test directory too #137

liamhuber opened this issue Dec 17, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@liamhuber
Copy link
Member

@XzzX pointed out it doesn't actually parse the tests directory! It's just here to improve readability, so IMO let's make the test suite more readable too!

This is low priority for me, but I am happy to make the PR -- should just be extending the path variable here. In the meantime I'll open it immediately as an issue to make room for any dissent.

@liamhuber liamhuber added the enhancement New feature or request label Dec 17, 2024
@liamhuber liamhuber self-assigned this Dec 17, 2024
@XzzX
Copy link
Contributor

XzzX commented Dec 18, 2024

@XzzX pointed out it doesn't actually parse the tests directory! It's just here to improve readability, so IMO let's make the test suite more readable too!

This is low priority for me, but I am happy to make the PR -- should just be extending the path variable here. In the meantime I'll open it immediately as an issue to make room for any dissent.

Oh no. I thought this is a trivial 5 minute change. Unfortunately, this is in the global actions repo... That complicates things.

@XzzX
Copy link
Contributor

XzzX commented Dec 18, 2024

Let's check the feedback #138 🙈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants