Skip to content

Motor restore script argparse #427

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

Merged
merged 11 commits into from
Jun 18, 2025
Merged

Conversation

rerpha
Copy link
Contributor

@rerpha rerpha commented Apr 28, 2025

Description of work

Add your own description here

To test

Which ticket does this PR fix?

Acceptance criteria

List the acceptance criteria for the PR


Code Review

  • Is the code of an acceptable quality?
  • Has the author taken into account the multi-threaded nature of the code?
  • Have the changes been recorded appropriately in a PR for release notes?
  • Has the manual system tests spreadsheet been updated?

Functional Tests

  • Do changes function as described? Add comments below that describe the tests performed.

Final steps

  • Reviewer has updated the submodule in the main EPICS repo? See Reviewing work for the subModules of EPICS in the Git workflow page for details.
  • Reviewer has merged the associated PR for the release notes

@rerpha rerpha marked this pull request as ready for review June 18, 2025 10:33
@rerpha rerpha moved this to Flash Review in Tasks Jun 18, 2025
@rerpha rerpha added this to Tasks Jun 18, 2025
@Tom-Willemsen Tom-Willemsen merged commit cffec40 into master Jun 18, 2025
5 checks passed
@Tom-Willemsen Tom-Willemsen deleted the motor_restore_script_argparse branch June 18, 2025 12:57
@Tom-Willemsen Tom-Willemsen removed this from Tasks Jun 18, 2025
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