Skip to content
This repository has been archived by the owner on Mar 18, 2024. It is now read-only.

fix(dependency-analysis): handle case where dependencies field is undefined #802

Merged
merged 1 commit into from
Jan 10, 2022

Conversation

aly76
Copy link
Contributor

@aly76 aly76 commented Jan 9, 2022

Handle case where dependencies field of an unlocked package is undefined, which
was causing an error when attempting to reference a method 'find' on type undefined.

…efined

Handle case where dependencies field on an unlocked package is undefined, which
was causing an error when attempting to reference a method 'find' on
dependencies.
@aly76 aly76 added the bug Something isn't working label Jan 9, 2022
@aly76 aly76 requested a review from azlam-abdulsalam January 9, 2022 23:58
@codecov-commenter
Copy link

Codecov Report

Merging #802 (09fbdcb) into develop (33d2dce) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #802   +/-   ##
========================================
  Coverage    66.22%   66.22%           
========================================
  Files           46       46           
  Lines         1593     1593           
  Branches       341      341           
========================================
  Hits          1055     1055           
  Misses         538      538           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5a69c21...09fbdcb. Read the comment docs.

@aly76 aly76 merged commit f66b7a2 into develop Jan 10, 2022
@aly76 aly76 deleted the fix/undefined-dependencies branch January 10, 2022 05:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants