Skip to content
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

CI: Remove superfluous step to publish Windows Qt artifacts #273

Closed

Conversation

PatTheMav
Copy link
Member

Description

Removes the publish step from the Windows Qt6 build job.

Motivation and Context

Publish step was not intended to be contained in the build job and is a left-over from when the job was copied over from the main workflow.

How Has This Been Tested?

Will need testing on CI.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@RytoEX
Copy link
Member

RytoEX commented Jan 27, 2025

Is it not needed so that the Qt WoA job has host components?

@PatTheMav
Copy link
Member Author

Is it not needed so that the Qt WoA job has host components?

D'oh, you're right. I just spotted those two artefacts (and no others) and was confused by that.

@PatTheMav PatTheMav closed this Jan 27, 2025
@PatTheMav PatTheMav deleted the ci-remove-nightly-artefacts branch January 27, 2025 17:31
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