File tree 6 files changed +10
-10
lines changed
6 files changed +10
-10
lines changed Original file line number Diff line number Diff line change 9
9
reasons] .
10
10
- Don't use the ` jQuery ` or ` $ ` global. Access jQuery via ` angular.element ` .
11
11
12
- [ avoid manual dependency annotations ] : http://robots. thoughtbot.com/avoid-angularjs-dependency-annotation-with-rails
12
+ [ avoid manual dependency annotations ] : http://thoughtbot.com/blog /avoid-angularjs-dependency-annotation-with-rails
13
13
[ ngannotate ] : https://github.com/kikonen/ngannotate-rails
14
14
[ performance reasons ] : https://github.com/angular-translate/angular-translate/wiki/Getting-Started#using-translate-directive
Original file line number Diff line number Diff line change 6
6
mailer view before merging. Use [ MailView] gem unless using Rails version
7
7
4.1.0 or later.
8
8
9
- [ amazon ses ] : http ://robots. thoughtbot.com/post/3105121049/delivering -email-with-amazon-ses-in-a-rails-3 -app
9
+ [ amazon ses ] : https ://thoughtbot.com/blog/deliver -email-with-amazon-ses-in-a-rails-app
10
10
[ sendgrid ] : https://devcenter.heroku.com/articles/sendgrid
11
11
[ mailview ] : https://github.com/37signals/mail_view
12
12
[ actionmailer preview ] : http://api.rubyonrails.org/v4.1.0/classes/ActionMailer/Base.html#class-ActionMailer::Base-label-Previewing+emails
Original file line number Diff line number Diff line change 10
10
exceeds 100 lines, it may be doing too many things.
11
11
- [ Tell, don't ask] .
12
12
13
- [ tell, don't ask ] : https://robots. thoughtbot.com/tell-dont-ask
13
+ [ tell, don't ask ] : https://thoughtbot.com/blog /tell-dont-ask
Original file line number Diff line number Diff line change 75
75
- [ Add foreign key constraints] in migrations.
76
76
77
77
[ `on_delete` behavior for foreign keys ] : http://api.rubyonrails.org/classes/ActiveRecord/ConnectionAdapters/SchemaStatements.html#method-i-add_foreign_key
78
- [ add foreign key constraints ] : http://robots. thoughtbot.com/referential-integrity-with-foreign-keys
78
+ [ add foreign key constraints ] : http://thoughtbot.com/blog /referential-integrity-with-foreign-keys
79
79
80
80
## Routes
81
81
99
99
- Use the user's name in the ` From ` header and email in the ` Reply-To ` when
100
100
[ delivering email on behalf of the app's users] .
101
101
102
- [ delivering email on behalf of the app's users ] : http://robots. thoughtbot.com/post/3215611590/recipe- delivering-email-on-behalf-of-users
102
+ [ delivering email on behalf of the app's users ] : http://thoughtbot.com/blog/ post/delivering-email-on-behalf-of-users
103
103
104
104
## Code Review
105
105
Original file line number Diff line number Diff line change 67
67
- Use an [ exact version] in the ` Gemfile ` for fragile gems, such as Rails.
68
68
69
69
[ ruby version ] : http://bundler.io/v1.3/gemfile_ruby.html
70
- [ exact version ] : http://robots. thoughtbot.com/post/35717411108 /a-healthy-bundle
71
- [ pessimistic version ] : http://robots. thoughtbot.com/post/35717411108 /a-healthy-bundle
72
- [ versionless ] : http://robots. thoughtbot.com/post/35717411108 /a-healthy-bundle
70
+ [ exact version ] : http://thoughtbot.com/blog /a-healthy-bundle
71
+ [ pessimistic version ] : http://thoughtbot.com/blog /a-healthy-bundle
72
+ [ versionless ] : http://thoughtbot.com/blog /a-healthy-bundle
73
73
74
74
## Ruby Gems
75
75
Original file line number Diff line number Diff line change 36
36
[ `allow` syntax ] : https://github.com/rspec/rspec-mocks#method-stubs
37
37
[ dependency injection ] : http://en.wikipedia.org/wiki/Dependency_injection
38
38
[ `delayed::job` matcher ] : https://gist.github.com/3186463
39
- [ stubs and spies ] : http://robots. thoughtbot.com/post/159805295 /spy-vs-spy
39
+ [ stubs and spies ] : http://thoughtbot.com/blog /spy-vs-spy
40
40
[ assertions about state ] : https://speakerdeck.com/skmetz/magic-tricks-of-testing-railsconf?slide=51
41
- [ fake ] : http://robots. thoughtbot.com/post/219216005 /fake-it
41
+ [ fake ] : http://thoughtbot.com/blog /fake-it
42
42
[ sut ] : http://xunitpatterns.com/SUT.html
43
43
44
44
## Acceptance Tests
You can’t perform that action at this time.
0 commit comments