search

ManageIQ / manageiq-schema

SQL schema and database migrations for ManageIQ
Apache License 2.0
20 stars 125 forks source link

Remove code for unsupported rails versions #736

Closed jrafanie closed 4 months ago

jrafanie commented 4 months ago

We already dropped support for rails 6 and lower so these conditionals are not needed.

Discovered when working on rails 7 support in: https://github.com/ManageIQ/manageiq-schema/pull/715

NOTE1: Hiding whitespace helps. NOTE2: This is completely standalone from #715.

miq-bot commented 4 months ago

Checked commit https://github.com/jrafanie/manageiq-schema/commit/ee852a06735fd78e30887c2d6cf3c9c72c710245 with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 1 file checked, 0 offenses detected Everything looks fine. :+1:

jrafanie commented 4 months ago

This is pretty straight forward. We haven't supported schema repo with rails versions less than 6 so I'm dropping that unused code here.