Skip to content

fix: Use correct env vars for git config in third-party notices job#2768

Merged
nr-nithya-santhosh merged 1 commit intoreleasefrom
fix/third-party-notices-git-config
Jan 23, 2026
Merged

fix: Use correct env vars for git config in third-party notices job#2768
nr-nithya-santhosh merged 1 commit intoreleasefrom
fix/third-party-notices-git-config

Conversation

@praveenkatha-nr
Copy link
Collaborator

@praveenkatha-nr praveenkatha-nr commented Jan 23, 2026

The workflow was referencing BOT_EMAIL and BOT_NAME which don't exist, instead of THIRD_PARTY_GIT_AUTHOR_EMAIL and THIRD_PARTY_GIT_AUTHOR_NAME which are defined in the workflow's env block.

Summary

A concise description of the changes being introduced. Please review the pre-merge checklist section to validate this pull request is ready for review and merge. If it is not ready, please mark the pull request as a draft.

The owners of this repo are not experts in the subject matter of the quickstarts. We review for the quickstart to be functional and for security risks. If you are seeking feedback on the content of the quickstart, please seek out a subject matter expert. If you are not an internal NR contributor, we can do our best to connect you with a content reviewer.

Pre merge checklist

  • Did you check you NRQL syntax? - Does it work?
  • Did you include a Data source and Documentation reference?
  • Are all documentation links publicly accessible?
  • Did you check your descriptive content for voice and tone?
  • Did you check your descriptive content for spelling and grammar errors?
  • Did you review your content with a subject matter expert? (e.g. a Browser agent quickstart is reviewed with a member of the Browser Agent team)

Dashboards

  • Does the PR contain a screenshot for each of your dashboards?
  • Do your screenshots show data?
  • Has the sanitization script been run on each dashboard?

Alerts

  • Did you check that your alerts actually work?
  • Are you trying to create standalone alerts? Standalone alerts are deprecated. They should only be included in quickstarts.

@praveenkatha-nr @claude
fix: use correct env vars for git config in third-party notices job
76f6e6b 
The workflow was referencing BOT_EMAIL and BOT_NAME which don't exist,
instead of THIRD_PARTY_GIT_AUTHOR_EMAIL and THIRD_PARTY_GIT_AUTHOR_NAME
which are defined in the workflow's env block.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@nr-nithya-santhosh nr-nithya-santhosh merged commit 60d0f75 into release Jan 23, 2026
13 checks passed
@nr-nithya-santhosh nr-nithya-santhosh deleted the fix/third-party-notices-git-config branch January 23, 2026 11:26
nr-nithya-santhosh added a commit that referenced this pull request Jan 23, 2026
@nr-nithya-santhosh @dependabot
@praveenkatha-nr @shashank-reddy-nr @claude
Release 2025-01-23 (#2769)
cb63788 
* chore(deps): bump form-data from 3.0.3 to 3.0.4 in /utils

Bumps [form-data](https://github.com/form-data/form-data) from 3.0.3 to 3.0.4.
- [Release notes](https://github.com/form-data/form-data/releases)
- [Changelog](https://github.com/form-data/form-data/blob/master/CHANGELOG.md)
- [Commits](form-data/form-data@v3.0.3...v3.0.4)

---
updated-dependencies:
- dependency-name: form-data
  dependency-version: 3.0.4
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore(deps): bump brace-expansion from 1.1.11 to 1.1.12 in /utils

Bumps [brace-expansion](https://github.com/juliangruber/brace-expansion) from 1.1.11 to 1.1.12.
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@1.1.11...v1.1.12)

---
updated-dependencies:
- dependency-name: brace-expansion
  dependency-version: 1.1.12
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore(deps): bump tmp and @inquirer/prompts in /utils

Removes [tmp](https://github.com/raszi/node-tmp). It's no longer used after updating ancestor dependency [@inquirer/prompts](https://github.com/SBoudrias/Inquirer.js). These dependencies need to be updated together.


Removes `tmp`

Updates `@inquirer/prompts` from 3.3.0 to 8.2.0
- [Release notes](https://github.com/SBoudrias/Inquirer.js/releases)
- [Commits](https://github.com/SBoudrias/Inquirer.js/compare/@inquirer/prompts@3.3.0...@inquirer/prompts@8.2.0)

---
updated-dependencies:
- dependency-name: tmp
  dependency-version: 
  dependency-type: indirect
- dependency-name: "@inquirer/prompts"
  dependency-version: 8.2.0
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore(deps): bump qs and express in /utils

Bumps [qs](https://github.com/ljharb/qs) to 6.14.1 and updates ancestor dependency [express](https://github.com/expressjs/express). These dependencies need to be updated together.


Updates `qs` from 6.13.0 to 6.14.1
- [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md)
- [Commits](ljharb/qs@v6.13.0...v6.14.1)

Updates `express` from 4.21.2 to 4.22.1
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/v4.22.1/History.md)
- [Commits](expressjs/express@4.21.2...v4.22.1)

---
updated-dependencies:
- dependency-name: qs
  dependency-version: 6.14.1
  dependency-type: indirect
- dependency-name: express
  dependency-version: 4.22.1
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: Fix OTel Kafka dashboard (#2766)

* fix: kafka dashboard

* fix: kafka dashboard

* fix: use correct env vars for git config in third-party notices job (#2768)

The workflow was referencing BOT_EMAIL and BOT_NAME which don't exist,
instead of THIRD_PARTY_GIT_AUTHOR_EMAIL and THIRD_PARTY_GIT_AUTHOR_NAME
which are defined in the workflow's env block.

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: praveenkatha-nr <pkatha@newrelic.com>
Co-authored-by: Pulipelly Shashank Reddy <spulipelly@newrelic.com>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
nr-nithya-santhosh added a commit that referenced this pull request Jan 27, 2026
@nr-nithya-santhosh @dependabot
@praveenkatha-nr @shashank-reddy-nr @claude
Release 2025-01-27 (#2773)
dde90e9 
* chore(deps): bump form-data from 3.0.3 to 3.0.4 in /utils

Bumps [form-data](https://github.com/form-data/form-data) from 3.0.3 to 3.0.4.
- [Release notes](https://github.com/form-data/form-data/releases)
- [Changelog](https://github.com/form-data/form-data/blob/master/CHANGELOG.md)
- [Commits](form-data/form-data@v3.0.3...v3.0.4)

---
updated-dependencies:
- dependency-name: form-data
  dependency-version: 3.0.4
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore(deps): bump brace-expansion from 1.1.11 to 1.1.12 in /utils

Bumps [brace-expansion](https://github.com/juliangruber/brace-expansion) from 1.1.11 to 1.1.12.
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@1.1.11...v1.1.12)

---
updated-dependencies:
- dependency-name: brace-expansion
  dependency-version: 1.1.12
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore(deps): bump tmp and @inquirer/prompts in /utils

Removes [tmp](https://github.com/raszi/node-tmp). It's no longer used after updating ancestor dependency [@inquirer/prompts](https://github.com/SBoudrias/Inquirer.js). These dependencies need to be updated together.


Removes `tmp`

Updates `@inquirer/prompts` from 3.3.0 to 8.2.0
- [Release notes](https://github.com/SBoudrias/Inquirer.js/releases)
- [Commits](https://github.com/SBoudrias/Inquirer.js/compare/@inquirer/prompts@3.3.0...@inquirer/prompts@8.2.0)

---
updated-dependencies:
- dependency-name: tmp
  dependency-version: 
  dependency-type: indirect
- dependency-name: "@inquirer/prompts"
  dependency-version: 8.2.0
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore(deps): bump qs and express in /utils

Bumps [qs](https://github.com/ljharb/qs) to 6.14.1 and updates ancestor dependency [express](https://github.com/expressjs/express). These dependencies need to be updated together.


Updates `qs` from 6.13.0 to 6.14.1
- [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md)
- [Commits](ljharb/qs@v6.13.0...v6.14.1)

Updates `express` from 4.21.2 to 4.22.1
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/v4.22.1/History.md)
- [Commits](expressjs/express@4.21.2...v4.22.1)

---
updated-dependencies:
- dependency-name: qs
  dependency-version: 6.14.1
  dependency-type: indirect
- dependency-name: express
  dependency-version: 4.22.1
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: Fix OTel Kafka dashboard (#2766)

* fix: kafka dashboard

* fix: kafka dashboard

* fix: use correct env vars for git config in third-party notices job (#2768)

The workflow was referencing BOT_EMAIL and BOT_NAME which don't exist,
instead of THIRD_PARTY_GIT_AUTHOR_EMAIL and THIRD_PARTY_GIT_AUTHOR_NAME
which are defined in the workflow's env block.

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

* fix: Fix otel kafka dashboard (#2770)

* fix: kafka dashboard

* fix: kafka dashboard

* fix: Removed Infra keyword from otel kafka DS

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: praveenkatha-nr <pkatha@newrelic.com>
Co-authored-by: Pulipelly Shashank Reddy <spulipelly@newrelic.com>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nr-nithya-santhosh nr-nithya-santhosh nr-nithya-santhosh approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@praveenkatha-nr @nr-nithya-santhosh

Comments