Skip to content

Conversation

@qnighy
Copy link
Collaborator

@qnighy qnighy commented May 18, 2021

Why

To support inheritance & mixin in the future, and just as a refactoring

What

Define a loader or a primary loader as a singleton method.

@codecov
Copy link

codecov bot commented May 18, 2021

Codecov Report

Merging #21 (fb28349) into master (ae04bfd) will increase coverage by 0.49%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   91.81%   92.30%   +0.49%     
==========================================
  Files           3        3              
  Lines         171      169       -2     
  Branches       38       37       -1     
==========================================
- Hits          157      156       -1     
  Misses          7        7              
+ Partials        7        6       -1     
Impacted Files Coverage Δ
lib/computed_model/dep_graph.rb 100.00% <100.00%> (ø)
lib/computed_model/model.rb 85.55% <100.00%> (+0.60%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae04bfd...fb28349. Read the comment docs.

@qnighy qnighy merged commit 21d2081 into master May 18, 2021
@qnighy qnighy deleted the qnighy/define-singleton-method branch May 18, 2021 03:36
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.

2 participants