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

Switch to GitHub Actions #13

Merged
merged 2 commits into from
Apr 1, 2022
Merged

Switch to GitHub Actions #13

merged 2 commits into from
Apr 1, 2022

Conversation

Fryguy
Copy link
Member

@Fryguy Fryguy commented Feb 15, 2022

@bdunne
Copy link
Member

bdunne commented Feb 22, 2022

Is this ready to go?

Copy link
Member

@kbrock kbrock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI all things

@Fryguy
Copy link
Member Author

Fryguy commented Feb 25, 2022

Is this ready to go?

I am review them one-by-one which is why I WIP'd them...I've found I need manual changes in most of them (even if they go green).

@Fryguy Fryguy changed the title [WIP] Switch to GitHub Actions Switch to GitHub Actions Mar 16, 2022
@Fryguy
Copy link
Member Author

Fryguy commented Mar 16, 2022

This is ready to go.

@miq-bot
Copy link
Member

miq-bot commented Mar 16, 2022

Checked commits Fryguy/manageiq-ssh-util@1e5aade~...a9ae525 with ruby 2.6.7, rubocop 1.19.1, haml-lint 0.35.0, and yamllint
4 files checked, 2 offenses detected

**

  • 💣 💥 🔥 🚒 - Linter/Rubocop - Linter::Rubocop STDERR:
/tmp/d20220316-8-1bmv568/.rubocop_local.yml: Warning: no department given for GlobalVars.

  • 💣 💥 🔥 🚒 - Linter/Yaml - missing config files

@@ -1,6 +1,7 @@
require 'net/ssh'
require 'net/sftp'
require 'tempfile'
require 'active_support'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm curious when/why this became a requirement, but rails/rails#43889 suggests that it's necessary.

Copy link
Member Author

@Fryguy Fryguy Mar 16, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's the one. It's also documented in the Rails docs that now you have to do require "active_support" first before you do any "partial" usage of "active_support". In this particular case, it happens to work anyway, but I want to future-proof the code in case they change something in the future.

@Fryguy Fryguy assigned agrare and unassigned bdunne Mar 31, 2022
@agrare agrare merged commit 9fd99aa into ManageIQ:master Apr 1, 2022
bdunne added a commit that referenced this pull request May 1, 2024
### Removed
- BREAKING: Drop support for Ruby 2.5 [#21]

### Changed
- Bump net-ssh for OpenSSL 3.0 support [#21]
- Update paambaati/codeclimate-action action to v6 [#22]
- Add renovate.json [#15]
- Update codeclimate channel to the latest in manageiq-style [#19]
- Test with ruby 3.1 and 3.0 [#18]
- Update actions/checkout version to v4 [#17]
- Update GitHub Actions versions [#16]
- Add timeout-minutes to setup-ruby job [#14]
- Switch to GitHub Actions, Handle newer version of activesupport [#13]
- Add .whitesource configuration file [#12]
- Update manageiq-style [#10]
- Add badges to readme [#9]
- Remove codeclimate-test-reporter [#8]
- Switch to manageiq-style [#7]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants