David Seddon
David Seddon
Nice idea. Happy to consider a pull request for this if anyone is interested. I suggest we follow the same approach as Ruff, that is we add a `--quiet` flag...
I don't understand, that seems to be what ruff is doing but I might be misunderstanding. In any case, feel free to outline here how you propose things should work...
> you might consider changing the default behavior but I totally understand if you don't want to I don't think I'll do this unless as part of a major release....
Thanks for raising. I'm assuming you're talking about errors that happen while running import linter (e.g. a malformed contract), rather than _contract failures_ which I think should just be included...
> I've turned copywriter into src.copywriter in my .import-linter file, and it seems to have fixed the problem, but I'm not a fan. I agree, I don't think that's the...
Thanks for the bug report - I'll look into it. Incidentally this should just have better error handling, as using the root package as a layer isn't valid.
> Is it a bug or just unsupported? Unsupported...or, maybe, meaningless. Each layer is (potentially) a subpackage, so a layer shouldn't contain another layer in the same contract. My expectation...
Thanks Ben for these suggestions. I agree we should support this somehow. I'm leaning towards adding an `as_packages` flag which, if it was set to false, would allow you to...
Having had more time to mull this over, I think it would the `as_packages` flag would be a good addition. Let me know if you would be interested in submitting...
Thanks for raising the issue! Seems like a good idea, happy to consider a pull request for this.