Skip to content

Conversation

@LowriJenkins
Copy link
Contributor

Description of work

Add a check when adding macros to the blockserver configs to not create them if the gui passes them with the useDefault value set to true. And run ruff format and check.

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

@LowriJenkins LowriJenkins force-pushed the Ticket8445_Handle_removal_of_defaults_from_config branch 5 times, most recently from 075b0f9 to 6795375 Compare September 17, 2025 14:32
@LowriJenkins LowriJenkins force-pushed the Ticket8445_Handle_removal_of_defaults_from_config branch from 936545d to ebd365c Compare February 9, 2026 11:14
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