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

Allow GH actions and drop travis #357

Merged
merged 3 commits into from
Aug 27, 2020

Conversation

ezr-ondrej
Copy link
Member

@ezr-ondrej ezr-ondrej commented Aug 3, 2020

  • Fix ability to run rubocop locally in the repository.
  • Add GH actions tests
  • Drop Travis tests

Example of run: https://github.com/ezr-ondrej/foreman_ansible/actions

@ezr-ondrej
Copy link
Member Author

ezr-ondrej commented Aug 3, 2020

Hmm for some reason the REX feature is not in the database if the tests are being run through GH actions.
See the failure in: https://github.com/ezr-ondrej/foreman_ansible/runs/942223564?check_suite_focus=true
Any idea why @adamruzicka?

@adamruzicka
Copy link
Contributor

I think that's the reason why I had to add bundle exec rake db:test:prepare when writing the gh action for leapp plugin https://github.com/theforeman/foreman_leapp/blob/master/.github/workflows/ruby.yml#L79 .

I think db:create creates development and test databases, migrate and seed prepare the development database. The when you run the tests, it clones the schema of the development database, but not the data inside it, so the db is completely blank. Or at least something along the lines of this

@ezr-ondrej
Copy link
Member Author

@adamruzicka
Copy link
Contributor

Could you rebase please?

@ezr-ondrej
Copy link
Member Author

Done 👍

@adamruzicka
Copy link
Contributor

I think rubocop is unhappy about this, unless codeclimate uses a different config 🤷

@ezr-ondrej
Copy link
Member Author

Hm, I've just turned that cop down as there is not a lot of certainity about it rubocop/rubocop-rails#260

@ezr-ondrej
Copy link
Member Author

I believe we are 💚 https://github.com/ezr-ondrej/foreman_ansible/actions (last two)

Copy link
Contributor

@adamruzicka adamruzicka left a comment

Choose a reason for hiding this comment

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

ACK

@adamruzicka adamruzicka merged commit 776e8a4 into theforeman:master Aug 27, 2020
@adamruzicka
Copy link
Contributor

Thank you @ezr-ondrej !

@ezr-ondrej ezr-ondrej deleted the gh_actions branch August 27, 2020 13:10
evgeni added a commit to evgeni/katello that referenced this pull request Jan 23, 2024
The cop is buggy before rubocop-rails 2.12.0 [1][2], but to get that
we'd need a newer theforeman-rubocop, which generates more offenses than
I am willing to fix today. Disable the cop like it was disabled in
foreman_ansible too [3].

[1] rubocop/rubocop-rails#260
[2] rubocop/rubocop-rails#526
[3] theforeman/foreman_ansible#357
jeremylenz pushed a commit to Katello/katello that referenced this pull request Jan 23, 2024
The cop is buggy before rubocop-rails 2.12.0 [1][2], but to get that
we'd need a newer theforeman-rubocop, which generates more offenses than
I am willing to fix today. Disable the cop like it was disabled in
foreman_ansible too [3].

[1] rubocop/rubocop-rails#260
[2] rubocop/rubocop-rails#526
[3] theforeman/foreman_ansible#357
m-bucher pushed a commit to ATIX-AG/katello that referenced this pull request Jun 21, 2024
The cop is buggy before rubocop-rails 2.12.0 [1][2], but to get that
we'd need a newer theforeman-rubocop, which generates more offenses than
I am willing to fix today. Disable the cop like it was disabled in
foreman_ansible too [3].

[1] rubocop/rubocop-rails#260
[2] rubocop/rubocop-rails#526
[3] theforeman/foreman_ansible#357

(cherry picked from commit 98c9fee)
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.

3 participants