Skip to content

docs: fix dbt package name in macros README (audit-hepler -> audit_helper) - #1512

Merged
gcko merged 2 commits into
DataRecce:mainfrom
vaibhav8a:fix-audit-helper-package-name
Aug 26, 2026
Merged

docs: fix dbt package name in macros README (audit-hepler -> audit_helper)#1512
gcko merged 2 commits into
DataRecce:mainfrom
vaibhav8a:fix-audit-helper-package-name

Conversation

@vaibhav8a

Copy link
Copy Markdown
Contributor

macros/README.md refers to the dbt package as audit-hepler - two issues in one token: a transposed hepler, and a hyphen where the package uses an underscore.

The package pinned in integration_tests/dbt/packages.yml is:

  - package: dbt-labs/audit_helper

so the correct name is audit_helper. That also matches dbt_profiler, which is named alongside it in the same sentence and already uses an underscore.

One-line docs fix.

@gcko
gcko self-requested a review August 24, 2026 02:36
@gcko gcko added the documentation Improvements or additions to documentation label Aug 24, 2026
@gcko

gcko commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

@vaibhav8a Thanks for catching and fixing this typo!

The DCO check is failing because the commit is missing a Signed-off-by line. Please run these commands from your PR branch:

git commit --amend --no-edit --signoff
git push --force-with-lease

The --signoff flag adds the required line using your configured Git name and email. Once you push the amended commit, the DCO check should run again.

Thanks again for the contribution!

The macros README referred to the package as `audit-hepler`. The actual
package, as pinned in integration_tests/dbt/packages.yml, is
dbt-labs/audit_helper - which also matches the underscore style of
dbt_profiler named alongside it in the same sentence.

Signed-off-by: Vaibhav Srivastava <vaibhavsri1712@gmail.com>
@vaibhav8a
vaibhav8a force-pushed the fix-audit-helper-package-name branch from c7d92da to d08a63a Compare August 25, 2026 09:19

@gcko gcko left a comment

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.

Looks good!

@gcko
gcko merged commit ccacb25 into DataRecce:main Aug 26, 2026
12 checks passed
@gcko

gcko commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Hi @vaibhav8a, thanks for this one. Good catch, and nice that you traced it back to packages.yml rather than just fixing the spelling.

Curious, are you running Recce on your own dbt project? If so I'd love to hear what you've been using it for and where it's gotten in your way. Happy to keep it here in the thread, or if you'd rather talk, I can set up a short call at a time that works in your timezone. We're a small team and hearing from people who actually use it helps us a lot.

Thanks again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants