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

fix: update github actions workflow to conditionally upload sdk artifacts #1562

Conversation

Junjiequan
Copy link
Contributor

Description

This PR includes upload-sdk-artifact workflow modification to conditionally upload SDK artifact and delete swagger-schema artifacts for pull request.

Motivation

Fixes

  • Bug fixed (#X)

Changes:

  • changes made

Tests included

  • Included for each change/fix?
  • Passing?

Documentation

  • swagger documentation updated (required for API changes)
  • official documentation updated

official documentation info

…acts and add cleanup step for non-push events
@Junjiequan Junjiequan changed the base branch from master to SWAP-4319-add-new-dataset-v4-minimal-controller December 16, 2024 12:00
@Junjiequan Junjiequan changed the title Fix GitHub workflow fix: update GitHub Actions workflow to conditionally upload SDK artifacts Dec 16, 2024
@martin-trajanovski martin-trajanovski changed the title fix: update GitHub Actions workflow to conditionally upload SDK artifacts fix: update github actions workflow to conditionally upload SDK artifacts Dec 16, 2024
@martin-trajanovski martin-trajanovski changed the title fix: update github actions workflow to conditionally upload SDK artifacts fix: update github actions workflow to conditionally upload sdk artifacts Dec 16, 2024
@Junjiequan Junjiequan merged commit 035ac08 into SWAP-4319-add-new-dataset-v4-minimal-controller Dec 16, 2024
12 of 13 checks passed
@Junjiequan Junjiequan deleted the fix-github-workflow branch December 16, 2024 12:10
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