- 08 Mar, 2020 1 commit
-
-
Andrew W. Lee authored
It was reported in https://github.com/ctran/annotate_models/issues/762 that column defaults were broken. This reverts changes made in #677 to restore the expected behavior of column defaults. For the time being columns with associated enums won't be working.
-
- 25 Feb, 2020 1 commit
-
-
Shu Fujita authored
This is the final completed version of refactoring AnnotateRoutes.
-
- 17 Feb, 2020 1 commit
-
-
Ryan authored
* Fix new lines after comments for rubocop compatability Fix #552, #607
-
- 13 Feb, 2020 2 commits
-
-
Shu Fujita authored
The test cases of AnnotateModels.get_schema_info was separated into two sections. So I merged them into one, and refactor test cases. This PR is new version of #741.
-
Shu Fujita authored
Refactor test cases for entire rewrite in the future.
-
- 06 Feb, 2020 1 commit
-
-
Shu Fujita authored
Refactor of test cases for AnnotateRoutes.remove_annotations after #736.
-
- 04 Feb, 2020 1 commit
-
-
Shu Fujita authored
Refactor test cases for AnnotateModels.get_schema_info with Globalize gem
-
- 27 Jan, 2020 1 commit
-
-
Shu Fujita authored
I fixed message to make them more natural English.
-
- 26 Jan, 2020 2 commits
-
-
Shu Fujita authored
Refactor RSpec for AnnotateModels (4) - AnnotateModels.get_schema_info (without custom options) (#735) I refactored test cases of AnnotateModels without custom options.
-
Shu Fujita authored
I refactored RSpec file for another large scale refactor in another PR.
-
- 24 Jan, 2020 1 commit
-
-
tvallois authored
The scope of this pull request is to allow annotate_models to generate models documentation using YARD. This is the first step, I'll add more features later.
-
- 20 Jan, 2020 3 commits
-
-
Shu Fujita authored
* Structuralize RSpec test cases of AnnotateModels.get_schema_info (with custom options) * Replace expression expansion to plain text * Refactor RSpec test cases of AnnotateModels.get_schema_info (with custom options) * Change position of test cases
-
Shu Fujita authored
-
Peter Gundel authored
* Add columns managed by Globalize gem Globalize hooks into the model and removes the translated columns from the `klass.columns`. This commit checks if globalize is hooked into the model and adds the necessary columns to the annotation array. * Disable Rubocop Metrics/BlockLength for spec files RSpec spec files can contain long blocks easily because of the outher describe methods. So this rule makes not too much sense for these files.
-
- 19 Jan, 2020 1 commit
-
-
Shu Fujita authored
I refactored and structuralized RSpec test cases of AnnotateModels for readability and scalability because it was too complex to read. cf. #718 In this PR, I refactored test cases of some methods in `AnnotateModels`. I will refactor test cases of other methods in another PR.
-
- 18 Jan, 2020 1 commit
-
-
Shu Fujita authored
I refactored `AnnotateRoutes.routes_exists?` and methods using this. The points are as follows. * Removing `puts` in `AnnotateRoutes.routes_exists?` * Using `File.exist?` instead of `File.exists?` because `File.exists?` is deprecated * Renaming `AnnotateRoutes.routes_exists?` to `AnnotateRoutes.routes_file_exists?` in order to make the name of method more explanatory
-
- 30 Dec, 2019 1 commit
-
-
Andrew W. Lee authored
This change converts .all_options into a constant and moves it into Annotate::Constants. It also changes usages of .all_options.
-
- 18 Dec, 2019 2 commits
-
-
Alexander Belozerov authored
-
inkstak authored
Same problem returned by @janosrusiczki in #430 Spatial columns with activerecord-postgis-adapter (5.2.2) are not displayed nice ``` # name :string(3) # active :boolean default(FALSE), not null # geom :geometry({:srid= geometry, 4326 ``` That's come from activerecord-postgis-adapter, with redefined columns as such : ``` #<ActiveRecord::ConnectionAdapters::PostGIS::SpatialColumn @sql_type="geometry(Geometry,4326)", @geo_type="Geometry", @geometric_type=RGeo::Feature::Geometry, @srid=4326, @limit={:srid=>4326, :type=>"geometry"} [...] > ``` This fix displays them like this : ``` # name :string(3) # active :boolean default(FALSE), not null # geometry :geometry geometry, 4326 ``` Another possibility would have been to display them as below, but it involves a lot of extra-spaces for other columns. ``` # name :string(3) # active :boolean default(FALSE), not null # geometry :geometry(Geometry, 4326) ```
-
- 09 Nov, 2019 4 commits
-
-
Alexander Belozerov authored
-
Alexander Belozerov authored
-
Kyle Zhao authored
-
hasimoto1009 authored
-
- 28 Sep, 2019 1 commit
-
-
Andrew W. Lee authored
Changed references to `--additional_file_patterns` to be `--additional-file-patterns` to make it consistent with other flags.
-
- 26 Sep, 2019 1 commit
-
-
Andrew W. Lee authored
Adds tests for `.include_routes?`, `.include_models?`, `.skip_on_migration?`. Also moves the `TRUE_RE` under the `Annotate::Constants` namespace.
-
- 03 Sep, 2019 1 commit
-
-
Andrew W. Lee authored
Prior to this change, `Annotate.include_models?` returned the inverse of `Annotate.include_routes?`. This made it so annotating models and routes was not possible to do together. This PR adds an explicit `--models` flag and also adds it the option to `lib/generators/annotate/templates/auto_annotate_models.rake` with the default being set to `false`. Fixes #563 and undoes the bug introduced in #485.
-
- 02 Sep, 2019 1 commit
-
-
Andrew W. Lee authored
In looking into #563, I realized there are gaps in test coverage. For example, `bin/annotate` doesn't have any tests. Tidying the spec directory to make it easier to add tests in future work.
-