Skip to content

Refactor workflows to run tests on PRs#89

Merged
jakubno merged 1 commit intomainfrom
pr-tests
Apr 25, 2025
Merged

Refactor workflows to run tests on PRs#89
jakubno merged 1 commit intomainfrom
pr-tests

Conversation

@jakubno
Copy link
Copy Markdown
Member

@jakubno jakubno commented Apr 25, 2025

Description

Run tests for all PRs

@jakubno jakubno added the improvement Improvement for current functionality label Apr 25, 2025
@jakubno jakubno self-assigned this Apr 25, 2025
IS_RELEASE=$(./.github/scripts/is_release_for_package.sh "@e2b/code-interpreter-template")
echo "release=$IS_RELEASE" >> "$GITHUB_OUTPUT"

charts-tests:
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this isn't needed, because it will run already in PR

if: always() &&
!contains(needs.*.result, 'failure') &&
!contains(needs.*.result, 'cancelled') &&
(needs.changes.outputs.python == 'true' || needs.changes.outputs.template == 'true')
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this isn't needed, because it will run already in PR

if: always() &&
!contains(needs.*.result, 'failure') &&
!contains(needs.*.result, 'cancelled') &&
(needs.changes.outputs.js == 'true' || needs.changes.outputs.template == 'true')
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this isn't needed, because it will run already in PR

@jakubno jakubno enabled auto-merge (squash) April 25, 2025 12:13
@jakubno jakubno merged commit d4f1d0b into main Apr 25, 2025
4 checks passed
@jakubno jakubno deleted the pr-tests branch April 25, 2025 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement for current functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants