-
Notifications
You must be signed in to change notification settings - Fork 897
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
Add unit to Instrument selection criteria #3184
Conversation
@asafm please sign the CLA so these changes can be considered for merging. |
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.
LGTM with a changelog entry + clear CLA.
@reyang Added line to changelog - I linked the PR, wasn't sure if I'm suppose to link the PR or the issue |
@asafm The CLA check is missing. Could you add that ? |
Working my company to get them to sign and add me |
@cijothomas CLA approved |
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.
LGTM.
left a non-blocking comment about the compliance matrix getting too big, so suggest avoid adding an entry there.
cc @jmacd |
Please rebase @asafm |
@bogdandrutu done |
Changes
Added unit criteria to the instrument selection criteria of a view.
Please provide a brief description of the changes here.
For non-trivial changes, follow the change proposal process and link to the related issue(s) and/or OTEP(s), update the
CHANGELOG.md
, and also be sure to updatespec-compliance-matrix.md
if necessary.Related issues
Fixes #3101
Related OTEP(s)
none