Skip to content

PackageJson Support No Error Throwing#13

Merged
coltenkrauter merged 1 commit intomainfrom
feature/coltenkrauter/2025-02-08/1
Feb 8, 2025
Merged

PackageJson Support No Error Throwing#13
coltenkrauter merged 1 commit intomainfrom
feature/coltenkrauter/2025-02-08/1

Conversation

@coltenkrauter
Copy link
Copy Markdown
Contributor

@coltenkrauter coltenkrauter commented Feb 8, 2025

Description

Support returning undefined when the path is not valid.

Type of Change

  • New feature
  • Bug fix
  • Chore
  • Configuration
  • Documentation
  • Refactoring
  • Hotfix
  • Security
  • GitFlow

Testing

  • Executed locally
  • Unit tests
  • Integration tests
  • Load tests
  • PR Status checks
  • Workflow execution (against non-default branch)
  • Other (explain)
  • Not worth it (explain)
  • Not applicable

Checklist

[Not required to be completed, just check off what you've done.]

  • Commits are reasonably sized and the commit message describes the changes well.
  • I have made corresponding changes to the documentation.
  • I have added unit tests for changes to business logic.
  • I have emitted metrics so our team can monitor the health of this system related to these changes.
  • I have added logs to ensure debugging is straight-forward when issues arise.
  • I have considered failure modes (e.g. major dependency becomes unavailable, bad data passed in).
  • I have added a comment to this PR to discuss any concerns/risks of these changes. This might include effects on performance, new dependencies, changes in behavior, or difficult things to test.
  • I have considered whether some of these changes should go into a shared package to increase code reusability.

@coltenkrauter coltenkrauter self-assigned this Feb 8, 2025
@coltenkrauter coltenkrauter merged commit 38ecc9d into main Feb 8, 2025
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.

1 participant