fix(nvgpu): Missing error handling not returning #333

Merged

icex2 merged 2 commits from pr333 into master 2025-02-19 21:41:06 +03:00

Conversation 0 Commits 2 Files Changed 1

+3 -1

icex2 commented 2025-02-19 21:36:46 +03:00 (Migrated from github.com)

Copy Link

Copy Source

If fetching the configuration fails, don’t continue with
processing any non-existing data.

Stack created with Sapling. Best reviewed with ReviewStack.

• feat(nvgpu): Add —overwrite-exists flag for profile creation (#335)
• fix(nvgpu): Initialize value of parameter to fix INVALID_ARGUMENT (#334)
• -> fix(nvgpu): Missing error handling not returning (#333)

If fetching the configuration fails, don’t continue with processing any non-existing data. --- [//]: # (BEGIN SAPLING FOOTER) Stack created with [Sapling](https://sapling-scm.com). Best reviewed with [ReviewStack](https://reviewstack.dev/djhackersdev/bemanitools/pull/333). * #335 * #334 * __->__ #333

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

acio

bio2

bst

bug

Something isn't working

ddr

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

ezusb

feature

gitlab merge request

good first issue

Good for newcomers

has attachment

help wanted

Extra attention is needed

high priority

iidx

invalid

This doesn't seem right

jb

low priority

museca

popn

question

Further information is requested

sdvx

tech debt

wontfix

This will not be worked on

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

Max

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Max/djhackersdev_bemanitools#333