-
Notifications
You must be signed in to change notification settings - Fork 23
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
Test Request #18
Test Request #18
Conversation
why this failed - |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments
.github/workflows/qokit-package.yml
Outdated
branches: [ "main", "**" ] | ||
pull_request: | ||
branches: [ "main" ] | ||
#branches: [ "main", "**" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason the lines are commented out and not simply deleted?
branches: [ "main", "**" ] | ||
pull_request: | ||
branches: [ "main" ] | ||
#branches: [ "main", "**" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason the lines are commented out and not simply deleted?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I commented those lines to test the GitHub Action for publishing. I have
- reverted the commented lines for
- .github/workflows/qokit-package.yml
- .github/workflows/qokit-python-only.yml
Deleted pypi-test.yml and merged that code in pypi-test-publish.yml
Modified pypi-publish.yml to publish to pypi on release tag
I have updated codes and all actions are green
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thank you @rajgane07!
This to test GA action from feature branch do not approve