simplify ruff exclude list

[danieleades]

No description provided.

[CLAassistant]

All committers have signed the CLA.

[IvanIsCoding]

I am debating about merging this. In one hand it is less lines of code.

However, I had to read ruff's documentation to understand why you used extend-exclude instead of exclude. So I think extending the defaults from ruff is clever but harder to maintain. I will let @mtreinish weight in

[coveralls]

Pull Request Test Coverage Report for Build 7563110066

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.04%) to 95.959%

---

Totals
Change from base Build 7549921600:	0.04%
Covered Lines:	16030
Relevant Lines:	16705

---

💛 - Coveralls

[danieleades]

I am debating about merging this. In one hand it is less lines of code.

However, I had to read ruff's documentation to understand why you used extend-exclude instead of exclude. So I think extending the defaults from ruff is clever but harder to maintain. I will let @mtreinish weight in

I'm very sympathetic to this argument, i totally see where you're coming from.

That said, I note the following from the ruff docs on the 'exclude' config:

Note that you'll typically want to use extend-exclude to modify the excluded paths.

The items in the default exclude list are super common in the python ecosystem, so i feel that explicitly listing them is unnecessary boilerplate. Using 'extend exclude' instead allows you to document what's different between rustworkx and a 'typical' python library.

[mtreinish]

Personally I'm fine either way, my preference is along with Ivan's to be explicit in the config on all the paths to exclude because I don't know off the top of my head what ruff is excluding by default (I didn't even realize there were default exclude paths). But at the end of the day were talking about a super minor difference in the configuration and both are doing the right thing.

[IvanIsCoding]

I think it is fine to merge given we don’t change the list much

If ruff breaks us in the future this was the culprit