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

Remove Appveyor in favour of GitHub actions #3172

Closed
wants to merge 1 commit into from
Closed

Conversation

xzyfer
Copy link
Contributor

@xzyfer xzyfer commented Aug 23, 2021

Before going too far down this rabbit hole, @nschonni do you think this makes sense?

@nschonni
Copy link
Contributor

I haven't looked in a bit, but Actions didn't have a way of using x86 Node.js before. That's the main reason it's still in the matrix. I think support landed for Azure DevOps, but not sure if Actions caught up yet

@xzyfer
Copy link
Contributor Author

xzyfer commented Aug 23, 2021

Thanks, that's the context I need :)

@nschonni
Copy link
Contributor

Looks like I started #3041, but didn't wrap it up. Not sure why anymore

@xzyfer
Copy link
Contributor Author

xzyfer commented Aug 23, 2021

Might have just been blocked on me missing the review request. Looks like these Windows x86 builds ran which is encouraging.

@nschonni
Copy link
Contributor

OK, I'll close this in favour of #3041

@nschonni nschonni closed this Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants