• Timothy Andrew's avatar
    Implement review comments from @dbalexandre. · 7b2ad2d5
    Timothy Andrew authored
    1. Remove `master_or_greater?` and `developer_or_greater?` in favor of
       `max_member_access`, which is a lot nicer.
    
    2. Remove a number of instances of `include Gitlab::Database::MigrationHelpers`
       in migrations that don't need this module. Also remove comments where
       not necessary.
    
    3. Remove duplicate entry in CHANGELOG.
    
    4. Move `ProtectedBranchAccessSelect` from Coffeescript to ES6.
    
    5. Split the `set_access_levels!` method in two - one each for `merge` and
       `push` access levels.
    7b2ad2d5