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

Draft: #271: Update Windows image from 2016 to 2019 #273

Closed
wants to merge 1 commit into from
Closed

Draft: #271: Update Windows image from 2016 to 2019 #273

wants to merge 1 commit into from

Conversation

swgnr
Copy link

@swgnr swgnr commented Sep 25, 2022

The CI pipeline is still using the windows-2016 image which has been phased out by GitHub earlier this year (see: actions/runner-images#4312). This causes no runners being available for picking up the win32 package/build steps, with them eventually timing out.

This initial PR bumps up the Windows image version used from 2016 to 2019. If things fail, we should now at least get an error message telling what's wrong.

Please do not merge yet. Thanks!

@codecov-commenter
Copy link

Codecov Report

Merging #273 (2b69a4d) into master (057f69f) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #273      +/-   ##
==========================================
- Coverage   93.80%   93.76%   -0.04%     
==========================================
  Files          23       23              
  Lines        2856     2870      +14     
==========================================
+ Hits         2679     2691      +12     
- Misses        177      179       +2     
Impacted Files Coverage Δ
lib/hci-socket/hci.js 99.13% <0.00%> (-0.28%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@rzr rzr changed the title #271: Update Windows image from 2016 to 2019 Draft: #271: Update Windows image from 2016 to 2019 Sep 25, 2022
@rzr
Copy link

rzr commented Sep 25, 2022

thx, to block merge i think there is a draft option you can set and unset from gh

@rzr
Copy link

rzr commented Oct 13, 2022

ping

@rzr
Copy link

rzr commented Oct 23, 2022

to unblock you, I'll release a version if this is fixed

@rzr
Copy link

rzr commented Nov 18, 2022

is it worth merging this draft ?

@rzr
Copy link

rzr commented Nov 21, 2022

please rebase

@rzr
Copy link

rzr commented Dec 17, 2022

can it be ready for next release before 2023 ?

@Debenben
Copy link

I included this commit here 51b5726 so this pull request is obsolete.

@rzr
Copy link

rzr commented Jan 3, 2023

please rebase to fix conflict

@Apollon77
Copy link

I think can be closed because done via #287

@rzr
Copy link

rzr commented Jan 10, 2023

Please rebase or close

@Apollon77
Copy link

@rzr simply close, this is already done if you check the github action which is currently in master

@rzr rzr closed this Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants