Skip to content

Commit f285f44

Browse files
committed
Fix remove_column() signature
This actually matters now that there's an "if exist" option
1 parent 5415be6 commit f285f44

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

lib/arjdbc/postgresql/adapter.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -503,7 +503,7 @@ def quote_table_name(name)
503503
alias_method :quote_schema_name, :quote_column_name
504504

505505
# Need to clear the cache even though the AR adapter doesn't for some reason
506-
def remove_column(table_name, column_name, type = nil, options = {})
506+
def remove_column(table_name, column_name, type = nil, **options)
507507
super
508508
clear_cache!
509509
end

lib/arjdbc/sqlite3/adapter.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ def add_column(table_name, column_name, type, **options) #:nodoc:
262262
end
263263
end
264264

265-
def remove_column(table_name, column_name, type = nil, options = {}) #:nodoc:
265+
def remove_column(table_name, column_name, type = nil, **options) #:nodoc:
266266
alter_table(table_name) do |definition|
267267
definition.remove_column column_name
268268
definition.foreign_keys.delete_if do |_, fk_options|

0 commit comments

Comments
 (0)