Skip to content

fix: normalize default values in schema for ruby 4.0 and rails 8 compatibility - #252

Open
matthv wants to merge 1 commit into
mainfrom
fix/normalize-default-value
Open

fix: normalize default values in schema for ruby 4.0 and rails 8 compatibility#252
matthv wants to merge 1 commit into
mainfrom
fix/normalize-default-value

Conversation

@matthv

@matthv matthv commented Jan 16, 2026

Copy link
Copy Markdown
Member

Definition of Done

General

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Test manually the implemented changes
  • Validate the code quality (indentation, syntax, style, simplicity, readability)

Security

  • Consider the security impact of the changes made
@qltysh

qltysh Bot commented Jan 16, 2026

Copy link
Copy Markdown

1 new issue

Tool Category Rule Count
qlty Structure Function with high complexity (count = 5): normalize_default_value 1

qltysh[bot]
qltysh Bot reviewed Jan 16, 2026
View reviewed changes
Comment thread ...st_admin_datasource_active_record/lib/forest_admin_datasource_active_record/parser/column.rb
column.default.to_f
else
column.default
end

@qltysh qltysh Bot Jan 16, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Function with high complexity (count = 5): normalize_default_value [qlty:function-complexity]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@matthv