[bugfix] Nullability Fix for HealthManager Argument

[tosettil-polimi]

Nullability Fix for HealthManager Argument

Issue

Issue #547 reported a nullability compatibility problem with the HealthManager.readWorkout method.

Problem Description

The method's NSNumber argument was missing an explicit nullability specification, which could cause compatibility issues with Android when using React Native.

Changes

• Added nonnull specification to the target parameter
• Ensures explicit nullability marking for better cross-platform compatibility

Code Modification

RCT_EXPORT_METHOD(captureRef:(nonnull NSNumber *)target withOptions:(NSDictionary *)options resolve:(RCTPromiseResolveBlock)resolve reject:(RCTPromiseRejectBlock)reject)

Impact

• Resolves React Native type compatibility warnings
• Improves type safety across platforms
• Ensures consistent behavior between iOS and Android

Testing

• Verified method signature works correctly with React Native
• Confirmed no breaking changes to existing functionality

[aymather]

Any idea when this might be merged?

:) <3

[tosettil-polimi]

Nope, see here the Issue for the temporary fix

[gaearon]

The PR title and description seem wrong (there is no "health manager" argument) but the fix is right.