I'm actually not sure adding the nixos options is a good idea. It's just feels like the start of a lot of maintenance work: why not add more and more options (eventually all)? For example, I personally use "about-filter", "source-filter" and "commit-filter", but this change only adds options for the first two. I guess I prefer to manage this directly in the transparent configText option.

@bjornfor wrote:

Please split into two commits, one to fix the cgit filters and the other to add the nixos options.

Sure, it's done.

I'm actually not sure adding the nixos options is a good idea. It's just feels like the start of a lot of maintenance work: why not add more and more options (eventually all)? For example, I personally use "about-filter", "source-filter" and "commit-filter", but this change only adds options for the first two. I guess I prefer to manage this directly in the transparent configText option.

I agree, the two configuration lines generated by the added options could alternatively be put as an example for the configText attribute. What do you think about it?

Yes, having those lines as examples is a good idea.

The 2nd commit should have prefix "nixos/cgit: [...]".

Filters moved to example and module commit reworded.

Thanks!