Skip to content

Conversation

@tahoemph
Copy link
Member

Summary

Implements ECOMM-7031 - Add custom RuboCop cops to Dutchie-Style gem

Changes:

  • Created unified LaunchDarkly Defaults cop that supports both:
    • Armageddon pattern: DutchieFeatureFlags.flag(), .context_flag(), etc.
    • MenuConnector pattern: ld_client.variation(), MenuConnector::App[:launchdarkly].variation()
  • Created MigrationSafetyAssured cop to flag questionable safety_assured usage in migrations
  • Bumped version to 2.1.0
  • Updated config and entry point to load both cops

Related tickets:

  • Epic: ECOMM-7030
  • Implementation: ECOMM-7031
  • Armageddon adoption: ECOMM-7032
  • MenuConnector adoption: ECOMM-7033

Test plan

  • Gem builds successfully
  • Cops load without errors
  • Manual testing in Armageddon after gem release
  • Manual testing in MenuConnector after gem release

Next steps

After this PR is merged and gem is released (v2.1.0):

  1. Update Armageddon to use new gem version (ECOMM-7032)
  2. Update MenuConnector to use new gem version (ECOMM-7033)
  3. Remove local cop implementations from both repos

- Create unified LaunchDarkly cop supporting both Armageddon and MenuConnector patterns
- Create MigrationSafetyAssured cop to flag questionable safety_assured usage
- Bump version to 2.1.0
- Update config and entry point
module Cop
module Dutchie
# Ensures all LaunchDarkly feature flag calls have default values
# to prevent failures during LaunchDarkly service outages
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

alexbeeken
alexbeeken previously approved these changes Dec 10, 2025
danhealy
danhealy previously approved these changes Dec 10, 2025
- Change from >= 3.1.4 to >= 3.0.6
- Allows MenuConnector (Ruby 3.0.6) to use this gem version
- Note: MenuConnector should still plan to upgrade Ruby eventually
@tahoemph tahoemph dismissed stale reviews from danhealy and alexbeeken via 956a4a8 December 10, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants