Struct not tuple for compiled per-file ignores

[carljm]

Summary

Code cleanup for per-file ignores; use a struct instead of a tuple.

Named the structs for individual ignores and the list of ignores CompiledPerFileIgnore and CompiledPerFileIgnoreList. Name choice is because we already have a PerFileIgnore struct for a pre-compiled-matchers form of the config. Name bikeshedding welcome.

Test Plan

Refactor, should not change behavior; existing tests pass.

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

[MichaReiser]

I like it :)

[BurntSushi]

LGTM!

[carljm]

@MichaReiser @snowsignal requesting another review because I would like to make sure that the change we made to display_settings! (to give it capability to display a GlobMatcher reasonably) looks OK.

The previous Display implementation here was actually pretty useless, it used {:#?} for a GlobMatcher which gives a massive output with all the internal structure of the GlobMatcher; suitable for debugging a GlobMatcher but not suitable for Display of the per-file-ignore settings at all. Given that nobody noticed that, I'm guessing we may never really use Display on these anywhere?

But anyway, the new version here is much better / more readable (for Display purposes), it just shows the actual file pattern of the GlobMatcher.

[snowsignal]

Looks good to me! Thank you 😁