-
Notifications
You must be signed in to change notification settings - Fork 105
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
Appveyor #116
Comments
I think we can remove it, I do not see the need to run both AppVeyor and GitHub Actions. |
I had already removed this a while ago from README and repo, but I do not have access to 3rd party integrations so you'll need to make these changes. |
@chmorgan This still needs to be removed. |
I do like keeping more than one option open. We use several for sharppcap,
GitHub workflows, Travis, circleci, azure, and Appveyor. They all offer
slightly different oses and tools revs. If it becomes a big burden we can
back away from appveyor, in the latest case it only took a minute to update
from vs2017 to vs2019 for core 3.1 support. So I’m inclined to say we
should keep it for now.
…On Sat, Jan 30, 2021 at 4:09 AM PhyxionNL ***@***.***> wrote:
@chmorgan <https://github.com/chmorgan> This still needs to be removed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#116 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJH4AGY6AAEPCHN6LG6DOTS4PEF5ANCNFSM4USYYGUQ>
.
|
I do not think we need to keep it here, nothing is using it. AppVeyor is also a hell to configure, is currently not working, and causes PRs checks to go red. |
Appveyor removed, closing. |
@chmorgan Can you try again to remove AppVeyor? It's still here and it keeps failing 🤔
|
hmm. let me look at that. |
How do you know that appveyor is failing? Are we getting emails from it? I see its still running somehow... |
alright, toggled off the packet net build setting. Let me know if you are still seeing failed builds from it. |
It's still running ☹ |
alright, toggled off the 'tests' setting too. I could just delete it entirely... |
Deleting would probably be better, as it's still going.
|
Create an
|
I'd rather see it removed completely, GH Actions works fine, I don't see a need to run multiple CIs here, especially when AppVeyor never works reliably 🙂 |
@PhyxionNL should we fix the appveyor build? Maybe we can just upgrade the visual studio version and it will be fixed? I can take a look.
The text was updated successfully, but these errors were encountered: