Skip to content

Conversation

@rerpha
Copy link
Contributor

@rerpha rerpha commented Jun 4, 2025

Description of work

Add your own description here

Ticket

Link to Ticket

Labels

Add appropriate label(s) to this PR

  • 'bluesky-Semver-Major' - Breaking Change
  • 'bluesky-Semver-Minor' - New feature
  • 'bluesky-bug' - Bug fix
  • 'bluesky-documentation' - Document Changes
  • 'bluesky-ignore-for-release' - Do not show in the release notes

Labels can be found of the right-hand sidebar of this PR once created

Acceptance criteria

List the acceptance criteria for the PR. The aim is provide information to help the reviewer

Documentation

Highlight and provide a link to any additions or changes to the documentation, if applicable. The aim is provide information to help the reviewer

@rerpha rerpha added the bluesky-ignore-for-release Non user-facing changes (e.g. CI setup) label Jun 4, 2025
### `ibex_bluesky_core` devices
- `BlockRw` and `SimpleDae`: `ibex_bluesky_core` as it's completely general / useful to all beamlines
- `ReflParameter`: in `ibex_bluesky_core.devices.reflectrometry` because it's useful across all reflectometers
- `ReflParameter`: in `ibex_bluesky_core.devices.reflectometry` because it's useful across all reflectometers
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we used the:

{py:obj}`ibex_bluesky_core.devices.reflectometry`

syntax sphinx would have caught this for us...

@Tom-Willemsen Tom-Willemsen merged commit 9b58361 into main Jun 4, 2025
13 checks passed
@Tom-Willemsen Tom-Willemsen deleted the rerpha-patch-1 branch June 4, 2025 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bluesky-ignore-for-release Non user-facing changes (e.g. CI setup) size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants