Skip to content

feat(wrapperModules.ripgrep): init#579

Open
Nadim147c wants to merge 1 commit into
BirdeeHub:mainfrom
Nadim147c:wrapperModules.ripgrep
Open

feat(wrapperModules.ripgrep): init#579
Nadim147c wants to merge 1 commit into
BirdeeHub:mainfrom
Nadim147c:wrapperModules.ripgrep

Conversation

@Nadim147c

Copy link
Copy Markdown

Added ripgrep module. I've added two options:

  • settings: attrset of options' names and values.
  • ignores: lines of glob for ripgrep ignore.

@Nadim147c
Nadim147c force-pushed the wrapperModules.ripgrep branch from e3fc55f to 0ab6952 Compare July 12, 2026 07:19

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In case you're interested, we have a dedicated test-lib to make writing tests more convenient:

https://birdeehub.github.io/nix-wrapper-modules/md/CONTRIBUTING.html#writing-tests

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was copying/following the yt-dlp tests. I will rewrite the checks.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I think those were written before the tlib was introduced. Unlucky :-)

@Nadim147c
Nadim147c force-pushed the wrapperModules.ripgrep branch from 0ab6952 to 9136b33 Compare July 13, 2026 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants