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

Re-enable functional tests in CI #522

Closed
wants to merge 10 commits into from
Closed

Conversation

mattbrictson
Copy link
Member

@mattbrictson mattbrictson commented Nov 17, 2023

Functional tests were disabled in CI in #523 due to GitHub Actions issues.

This PR plans to reenable functional tests in CI once those issues are resolved. It also cargo-cults some Vagrantfile improvements from https://github.com/fastapi-mvc/example/blob/6b29b3fe3dc9004aa7492c4d2ecd569fdf7e3ec3/Vagrantfile

@mattbrictson mattbrictson added the 🏠 Housekeeping Non-user facing cleanup and maintenance label Nov 17, 2023
@mattbrictson mattbrictson force-pushed the chores/fix-vagrant branch 4 times, most recently from ae5a887 to 0950025 Compare November 18, 2023 00:11
@mattbrictson mattbrictson deleted the chores/fix-vagrant branch November 18, 2023 00:14
@mattbrictson mattbrictson restored the chores/fix-vagrant branch November 22, 2023 18:26
@mattbrictson mattbrictson reopened this Nov 22, 2023
@mattbrictson mattbrictson changed the title WIP attempt to fix Vagrant CI failures Re-enable functional tests in CI Nov 22, 2023
@mattbrictson mattbrictson marked this pull request as ready for review December 22, 2023 17:01
@mattbrictson mattbrictson marked this pull request as draft December 22, 2023 17:07
Old Rubies such as Ruby 2.0 resolve a version of net-sftp that does not
have the `File#size` convenience method. To support these older
versions, use the lower level `File#stat`, which works for versions of
net-sftp all the way back to 2.1.2.

This fixes a `NoMethodError` when SFTP is used via sshkit on Ruby 2.0.
@mattbrictson
Copy link
Member Author

Superseded by #539

@mattbrictson mattbrictson deleted the chores/fix-vagrant branch July 3, 2024 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏠 Housekeeping Non-user facing cleanup and maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant