-
Notifications
You must be signed in to change notification settings - Fork 0
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
Ticket6056 add ioc attocube #1744
base: master
Are you sure you want to change the base?
Conversation
67e4949
to
a4fd053
Compare
Which is called by the pre-commit hook
a4fd053
to
d274020
Compare
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.
Just a couple of minor changes
<macros> | ||
<macro> | ||
<name>ATTOCUBE</name> | ||
<description>The ATTOCUBE PV prefix (e.g. ATTOCUBE_01)</description> |
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.
Should probably have a default of ATTOCUBE_01
on the macro
@@ -3938,6 +3938,21 @@ | |||
</categories> | |||
</value> | |||
</entry> | |||
<entry> | |||
<key>ATTOCUBE</key> |
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.
Think we usually have OPI names non-capitilised, so Attocube
ISISComputingGroup/IBEX#6056