Skip to content

Conversation

@lauramurgatroyd
Copy link
Member

@lauramurgatroyd lauramurgatroyd commented Nov 13, 2025

Added a new job 'conda-print' to print contents of the dist folder after building the conda package.

Describe your changes

Describe any testing you have performed

Consider adding example code to examples
Added conda-print job to CI workflow and from that I learnt that the path should just be noarch, not dist.

Link relevant issues

Checklist when you are ready to request a review

  • I have performed a self-review of my code
  • I have added docstrings in line with the guidance in the CIL developer guide
  • I have implemented unit tests that cover any new or modified functionality
  • CHANGELOG.md has been updated with any functionality change
  • Request review from all relevant developers
  • Change pull request label to 'waiting for review'

Contribution Notes

  • The content of this Pull Request (the Contribution) is intentionally submitted for inclusion in CILViewer (the Work) under the terms and conditions of the Apache-2.0 License
  • I confirm that the contribution does not violate any intellectual property rights of third parties

Qt contributions should follow Qt naming conventions i.e. camelCase method names.

VTK contributions should follow VTK naming conventions i.e. PascalCase method names.

Added a new job 'conda-print' to print contents of the dist folder after building the conda package.
Remove condition for conda-print job to always run.
Changed directory listing command to show current directory instead of 'dist'.
Change directory listing command to show 'noarch' folder contents.
@lauramurgatroyd lauramurgatroyd changed the title Fix conda-publish action Fix path to upload in conda-publish action Nov 13, 2025
@lauramurgatroyd lauramurgatroyd marked this pull request as ready for review November 13, 2025 14:23
@lauramurgatroyd lauramurgatroyd merged commit d52887a into master Nov 13, 2025
4 checks passed
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