24 Mar, 2014

3 commits

  • Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
    Dmitriy Zaporozhets
     
  • Removes the following from test output:
    
      DEPRECATION WARNING: It looks like you are eager loading table(s) (one
      of: merge_requests, projects) that are referenced in a string SQL
      snippet. For example:
    
          Post.includes(:comments).where("comments.title = 'foo'")
    
      Currently, Active Record recognizes the table in the string, and knows
      to JOIN the comments table to the query, rather than loading comments in
      a separate query. However, doing this without writing a full-blown SQL
      parser is inherently flawed. Since we don't want to write an SQL parser,
      we are removing this functionality. From now on, you must explicitly
      tell Active Record when you are referencing a table from a string:
    
          Post.includes(:comments).where("comments.title =
      'foo'").references(:comments)
    
      If you don't rely on implicit join references you can disable the
      feature entirely by setting
      `config.active_record.disable_implicit_join_references = true`.
    Robert Speicher
     
  • Mobile UI improvements
    Dmitriy Zaporozhets
     

23 Mar, 2014

1 commit


22 Mar, 2014

1 commit

  • The "Merged by" and "Closed by" statements end with an extra space followed by a period. This doesn't seem necessary to me and not having the extra space followed by a period looks better to me. Also, the top of the merge request doesn't have the sentence ending with a period.
    awiddersheim
     

21 Mar, 2014

1 commit


20 Mar, 2014

4 commits


19 Mar, 2014

7 commits


18 Mar, 2014

2 commits


17 Mar, 2014

5 commits


15 Mar, 2014

1 commit


14 Mar, 2014

11 commits


13 Mar, 2014

4 commits