Observers #5
Observers #5
Conversation
|
Changes Unknown when pulling 992f3e2530e931c13362eac3af26b977e895cdda on observers into ** on master**. |
|
Changes Unknown when pulling 62921e5 on observers into ** on master**. |
| @@ -0,0 +1,5 @@ | |||
| require 'rails_helper' | |||
|
|
|||
| describe <%= class_name %>Observer do | |||
There was a problem hiding this comment.
unterminated string meets end of file
| # before_create { puts "yes, you can provide a block to work with" } | ||
| # | ||
| # def run | ||
| # p object # object holds an <%= class_name %> instance. |
There was a problem hiding this comment.
unexpected token tIDENTIFIER
unterminated string meets end of file
| @@ -0,0 +1,8 @@ | |||
| class <%= class_name %>Observer < PowerTypes::Observer | |||
|
@ldlsegovia como recomendacion, yo haria este PR mas chico, solo con lo de observers... Pondria en otro PR lo de travis para el release y el readme arreglado. |
|
@blackjid con "el readme arreglado" te referis a los badges? porque el cambio grande que hice en el README es para agregar docs sobre los observers. |
|
Me parecio q el readme ademas de lo de observers tenia de todoa los otros tipos, que no estaban documentados. |
|
So pienso q es mas facil que el asigneeo mire cuando es mas chico y acotado a lo que realmente es el pr. En este caso pixar. Lo de los linters tambien lo tiraria a otro |
|
me parece! |
|
Sobre el README, en realidad lo reesctructuré para meter lo de los observers y para facilitar el agregado de nuevos tipos. Por esto, lo dejo en el PR de observers |
No description provided.