Skip to content

Conversation

@MialLewis
Copy link
Owner

Description of work

Closes #xxxx.

To test:


Reviewer

Your comments will be used as part of the gatekeeper process. Comment clearly on what you have checked and tested during your review. Provide an audit trail for any changes requested.

As per the review guidelines:

  • Is the code of an acceptable quality? (Code standards/GUI standards)
  • Has a thorough functional test been performed? Do the changes handle unexpected input/situations?
  • Are appropriately scoped unit and/or system tests provided?
  • Do the release notes conform to the guidelines and describe the changes appropriately?
  • Has the relevant (user and developer) documentation been added/updated?
  • If the PR author isn’t in the mantid-developers or mantid-contributors teams, add a review comment rerun ci to authorize/rerun the CI

Gatekeeper

As per the gatekeeping guidelines:

  • Has a thorough first line review been conducted, including functional testing?
  • At a high-level, is the code quality sufficient?
  • Are the base, milestone and labels correct?

SilkeSchomann pushed a commit to mantidproject/mantid that referenced this pull request Nov 20, 2025
### Description of work

A minimal addition of our Doxygen build test to GitHub actions.

As the logic is encapsulated within `Doxygen.sh`, any change to the
script (i.e using pixie > mamba) will be picked up automatically.

The workflow is likely to developed further as the migration to GHA
progresses, and as pixi is implemented.

Closes #40286

<!-- If issue raised by user. Do not leak email addresses.
**Report to:** [user name]
-->

### To test:

Check the relevant workflow has run on this PR.
Confirm workflow has run on this PR and failed, the change made is bad:
MialLewis#7

Co-authored-by: thomashampson <thomas.hampson@stfc.ac.uk>
peterfpeterson pushed a commit to peterfpeterson/mantid that referenced this pull request Nov 20, 2025
### Description of work

A minimal addition of our Doxygen build test to GitHub actions.

As the logic is encapsulated within `Doxygen.sh`, any change to the
script (i.e using pixie > mamba) will be picked up automatically.

The workflow is likely to developed further as the migration to GHA
progresses, and as pixi is implemented.

Closes mantidproject#40286

<!-- If issue raised by user. Do not leak email addresses.
**Report to:** [user name]
-->

### To test:

Check the relevant workflow has run on this PR.
Confirm workflow has run on this PR and failed, the change made is bad:
MialLewis#7

Co-authored-by: thomashampson <thomas.hampson@stfc.ac.uk>
adriazalvarez pushed a commit to mantidproject/mantid that referenced this pull request Dec 1, 2025
### Description of work

A minimal addition of our Doxygen build test to GitHub actions.

As the logic is encapsulated within `Doxygen.sh`, any change to the
script (i.e using pixie > mamba) will be picked up automatically.

The workflow is likely to developed further as the migration to GHA
progresses, and as pixi is implemented.

Closes #40286

<!-- If issue raised by user. Do not leak email addresses.
**Report to:** [user name]
-->

### To test:

Check the relevant workflow has run on this PR.
Confirm workflow has run on this PR and failed, the change made is bad:
MialLewis#7

Co-authored-by: thomashampson <thomas.hampson@stfc.ac.uk>
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