fix: Remove invalid prereq `check_variation_range` check

launchdarkly / ruby-server-sdk

LaunchDarkly Server-side SDK for Ruby

https://docs.launchdarkly.com/sdk/server-side/ruby

Other

34 stars 50 forks source link

fix: Remove invalid prereq `check_variation_range` check #261

Closed keelerm84 closed 5 months ago

keelerm84 commented 5 months ago

When parsing flag payload information, the SDK attempts to pre-check several failure conditions. One of those conditions was to ensure that a provided prereq has a valid variation index.

However, the system cannot actually perform this check at parse time since the prerequisite flag might not have been parsed yet.

As this check served only as a nice to have, I have removed it and added a test verifying the prereq behavior still operates as expected.

Fixes #260

shortcut-integration[bot] commented 5 months ago

This pull request has been linked to Shortcut Story #237484: Fix prereq range check.