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.

Sign up for GitHub

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

Jump to bottom

[CI] Use early exit for wasmer_init_works_1 and wasmer_init_works_2 when WAPM_DEV_TOKEN is empty #3733

Merged
merged 1 commit into from
Apr 3, 2023
Merged

[CI] Use early exit for wasmer_init_works_1 and wasmer_init_works_2 when WAPM_DEV_TOKEN is empty #3733

merged 1 commit into from
Apr 3, 2023

Conversation

ptitSeb
Copy link
Contributor

@ptitSeb ptitSeb commented Apr 3, 2023

WAPM_DEV_TOKEN can be empty for external contributors.
The tests wasmer_init_works_1 and wasmer_init_works_2 must exit in that case or they will fail.

@ptitSeb
[CI] Use early exit for wasmer_init_works_1 and wasmer_init_works_2 w…
e7c5da3 
…hen WAPM_DEV_TOKEN is empty (for external contributors)
@ptitSeb ptitSeb enabled auto-merge (squash) April 3, 2023 08:32
@ptitSeb ptitSeb merged commit 35beca3 into master Apr 3, 2023
@ptitSeb ptitSeb deleted the fix_ci_external branch April 3, 2023 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@theduke theduke theduke approved these changes

@Michael-F-Bryan Michael-F-Bryan Awaiting requested review from Michael-F-Bryan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ptitSeb @theduke