Skip to content

Conversation

@Sashwatdas123
Copy link
Contributor

@Sashwatdas123 Sashwatdas123 commented Aug 7, 2025

Ticket: NR-440836

Details:

  • Added support for Ingest-License Key.
  • Command supporting ingest-license key:
newrelic-lambda layers install \
    --function <Function Name> \
    --nr-account-id <Your Account Id> \
    --nr-ingest-key <Your Ingest Key>

@codecov
Copy link

codecov bot commented Aug 8, 2025

Codecov Report

❌ Patch coverage is 77.77778% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.04%. Comparing base (8ace961) to head (c7297b8).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
newrelic_lambda_cli/api.py 0.00% 2 Missing ⚠️
newrelic_lambda_cli/cli/layers.py 60.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #344      +/-   ##
==========================================
+ Coverage   76.26%   77.04%   +0.77%     
==========================================
  Files          18       18              
  Lines        1626     1642      +16     
==========================================
+ Hits         1240     1265      +25     
+ Misses        386      377       -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Sashwatdas123 Sashwatdas123 changed the title Added Support for ingest license key feat: Added Support for ingest license key Aug 8, 2025
Copy link
Contributor

@aavinash-nr aavinash-nr left a comment

Choose a reason for hiding this comment

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

@Sashwatdas123 Looks good to me 👏 . I have locally tested everything working fine.

@Sashwatdas123 Sashwatdas123 merged commit 8978c2d into newrelic:main Aug 8, 2025
7 checks passed
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