• Timothy Andrew's avatar
    Implement final review comments from @DouweM and @rymai · f2ed82fa
    Timothy Andrew authored
    - Have `Uniquify` take a block instead of a Proc/function. This is more
      idiomatic than passing around a function in Ruby.
    
    - Block a user before moving their issues to the ghost user. This avoids a data
      race where an issue is created after the issues are migrated to the ghost user,
      and before the destroy takes place.
    
    - No need to migrate issues (to the ghost user) in a transaction, because
      we're using `update_all`
    
    - Other minor changes
    f2ed82fa
user.rb 31.1 KB