• Timothy Andrew's avatar
    Implement final review comments from @DouweM and @rymai · 040eff3e
    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
    040eff3e
user.rb 32.6 KB