Removed unnecessary print statement in writeReg (Addresses #47) - Githubissues

cms-gem-daq-project / reg_utils

0 stars 9 forks source link

Removed unnecessary print statement in writeReg (Addresses #47) #48

Closed bdorney closed 5 years ago

bdorney commented 5 years ago

Description

Closes issue #47

Types of changes

[x] Non-breaking change (fix or feature that causes only cosmetic differences)

Breaking because it will no longer print at each write attempt.

Motivation and Context

See #47.

How Has This Been Tested?

Change was trivial.

Screenshots (if appropriate):

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

jsturdy commented 5 years ago

I actually don't think this is a breaking change, as described in the head of PR, as it only changes the (excessive) logging

Yeah... I modified the content and can probably update the PR templates with these options