-
Notifications
You must be signed in to change notification settings - Fork 28
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
Switch to GitHub Actions #178
Conversation
This one is going to be tough, because of the postgres-specific tests. |
15bd24b
to
e75c349
Compare
Postponing my work on this one...I think I mostly got the TTY problems fixed, through there's still a couple that don't seem to be honoring the newline The database backup/restore is gonna need a complete overhaul I think, which I don't want to spend time on at the moment. |
Rakefile
Outdated
end | ||
# task :default do | ||
# Rake::Task["spec#{':ci' if ENV['CI']}"].invoke | ||
# end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a hack to avoid the postgres tests and we should undo this.
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation. |
Checked commits Fryguy/manageiq-appliance_console@b6198dc~...48d0f9f with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint spec/spec_helper.rb
|
this was necessary to get older version of highline up and running in rails see ManageIQ#178 df5cffb
this was necessary to get older version of highline up and running in rails see ManageIQ#178 df5cffb
CI: https://github.com/Fryguy/manageiq-appliance_console/actions?query=branch%3Agithub_actions
To replicate the tty issues locally, run the tests with: