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: Migrate slow test suit from Travis CI #3450

Closed
wants to merge 3 commits into from

Conversation

jwagantall
Copy link

No description provided.

@jwagantall
Copy link
Author

Temporarily enabling pull request triggers in tests.yml for testing purposes

@@ -1,6 +1,6 @@
name: urchin tests

on: [push]
on: [pull_request, push]
Copy link
Member

Choose a reason for hiding this comment

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

this is actually a good fix and should have always been this way :-)

@ljharb ljharb marked this pull request as draft October 30, 2024 05:04
@ljharb ljharb added the testing Stuff related to testing nvm itself. label Oct 30, 2024
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>
@jwagantall jwagantall closed this by deleting the head repository Nov 15, 2024
@jwagantall
Copy link
Author

Sorry, I think my PR got a little messed up.. I had to create a new one #3470

@ljharb
Copy link
Member

ljharb commented Nov 15, 2024

in the future, i'm happy to help figure it out without deleting the repo and starting over.

@jwagantall
Copy link
Author

Thank you!.. actually it was really something basic but got distracted it and re-created my fork.. :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Stuff related to testing nvm itself.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants