Skip to content

0.2.1 - fixed ancestor walking on final instance attributes#7

Merged
bnlucas merged 2 commits intomainfrom
0.2.1
May 2, 2025
Merged

0.2.1 - fixed ancestor walking on final instance attributes#7
bnlucas merged 2 commits intomainfrom
0.2.1

Conversation

@bnlucas
Copy link
Owner

@bnlucas bnlucas commented May 2, 2025

  • Fixed an issue where only final: true instance variables defined on the current/class had their values applied.

    • Now walks the ancestor tree to ensure all attributes get set.
    module Options
      include Cattri
    
      cattri :enabled, true, final: true # wasn't being set previously
    end
    
    class Attribute
      include Options
    
      def initialize(enabled: true)
        seld.enabled = enabled
      end
    end
  • Cleanup of cattri.gemspec and bin/console.

@codecov
Copy link

codecov bot commented May 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (02c6642) to head (b9aa5f1).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           15        15           
  Lines          363       364    +1     
  Branches        49        49           
=========================================
+ Hits           363       364    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bnlucas bnlucas merged commit 7f535bd into main May 2, 2025
4 checks passed
@bnlucas bnlucas deleted the 0.2.1 branch May 2, 2025 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant