T0mstone

Results 49 comments of T0mstone

...or rather than that, I went with `SourceResult`.

Since we have Clippy in CI, maybe we should add `too_many_arguments = "allow"` to `Cargo.toml` instead of manually silencing it like I did now? This lint doesn't seem very productive...

In fact, there are 33 instances (32 without the newly added one) of `#[allow(clippy::too_many_arguments)]` in the repo.

Some thoughts: - What about infinity? Currently, the heading counter has infinite components and you could use a level-35123 heading if you wanted to (tho the output would be basically...

Alright, that's good to hear. In that case I have no further objections. The issue with insanely deep headings is probably best ignored for now. I don't think it's very...

> Because note that `heading` itself has "multiple levels". But this should really be allowed anymore. The individual units should be single-number. I disagree. In a call like `counter(a, b,...

How about letting the `numbering` argument itself be either a string or a dictionary `(pattern: str, prepend: array)`?

I'm not sure if 72c219d is the best fix for the new test failures, I just went for the most simple and easy fix of adding some useless content after...

Thanks for doing the work for me😅 I went with a `Span` as the argument for that helper function, since the other parts of the expr aren't used. Maybe this...

I like your idea (tho it should probably be called `sys.input-files` for consistency), but I just want to point out that using symlinks is easy and arguably more natural. OTOH,...