Skip to content

Update CI - Windows, forks, etc#85

Merged
aycabta merged 3 commits intoruby:masterfrom
MSP-Greg:actions-windows
Dec 18, 2019
Merged

Update CI - Windows, forks, etc#85
aycabta merged 3 commits intoruby:masterfrom
MSP-Greg:actions-windows

Conversation

@MSP-Greg
Copy link
Contributor

@MSP-Greg MSP-Greg commented Dec 11, 2019

This PR only affects CI.

  1. Rework Windows CI -
    1.1 change script that copies ruby/ruby files from a .bat script to a .ps1 script (PowerShell)
    1.2 rename readline.so file
    1.3 use an 'action' that installs current Rubies, versus the old Rubies on Actions.
    1.4 move master/trunk testing to GitHub Actions ('9.9.x')

  2. Allow CI to run in forks on branches other than 'master'

  3. Split ci-test and test in Rakefile and workflows

Notes:

  1. Windows Rubies 2.5 & 2.6 contain a readline.so file, but do not contain the dll they need to load.

  2. Left AppVeyor CI as is.

  3. To allow Ruby master to always work, it is versioned as '9.9.x' in the workflow matrix, and installed to the 'C:/Ruby99-x64' folder. When 2.7.0 release is available, I'll update the 'action'.

1. Uses current Ruby builds, also master
2. Changes download-test_readline.bat to a PowerShell script
3. PowerShell script also renames readline.so to readline.no
Current workflows restrict branch pushes to 'master', hinders contributors working in forks.
@MSP-Greg
Copy link
Contributor Author

@aycabta

I continued with another branch, replaced the download-test_readline.* files with one Ruby file. The script spins up four connections and downloads the files from ruby/ruby, it also renames readline in all currently tested platforms so the only tests run with ci-test are reline tests.

Also, removed Appveyor. No need to wait if ruby/ruby is stacked up with CI there...

See last two commits at:
https://github.com/MSP-Greg/reline/tree/actions-windows-rb

@aycabta aycabta merged commit 5d0dd0c into ruby:master Dec 18, 2019
@aycabta
Copy link
Member

aycabta commented Dec 18, 2019

Thank you. I'll consider the Ruby script version after Ruby 2.7 is released...

@MSP-Greg MSP-Greg deleted the actions-windows branch February 17, 2020 15:12
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.

2 participants