-
Notifications
You must be signed in to change notification settings - Fork 582
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: Rename ProjectParameter to ProjectVersionParameter #170
Conversation
Codecov Report
@@ Coverage Diff @@
## main #170 +/- ##
==========================================
+ Coverage 65.54% 65.77% +0.22%
==========================================
Files 103 103
Lines 5294 5294
Branches 68 68
==========================================
+ Hits 3470 3482 +12
+ Misses 1495 1485 -10
+ Partials 329 327 -2
Continue to review full report at Codecov.
|
This was confusing with ParameterValue before. It still is a bit, but this should help distinguish scope.
95b1365
to
1af0b50
Compare
@@ -40,11 +40,11 @@ func Compute(ctx context.Context, db database.Store, scope Scope) ([]Value, erro | |||
compute := &compute{ | |||
db: db, | |||
computedParameterByName: map[string]Value{}, | |||
projectVersionParametersByName: map[string]database.ProjectParameter{}, | |||
projectVersionParametersByName: map[string]database.ProjectVersionParameter{}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating this name - it does help clarify the scope for me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for keeping the names up-to-date, LGTM 👍
d719383
to
2df0927
Compare
This was confusing with ParameterValue before. It still is a bit,
but this should help distinguish scope.