Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Silent errors #251

Closed
alepore opened this issue May 26, 2015 · 2 comments
Closed

Silent errors #251

alepore opened this issue May 26, 2015 · 2 comments
Assignees
Labels
Milestone

Comments

@alepore
Copy link

alepore commented May 26, 2015

With some gems like Spree we use lots of "decorators" files under app/models, which are not ruby classes but simple Model.class_eval do ... files.
Annotate gives lots of verbose warning like:

Unable to annotate app/models/spree/order_decorator.rb: cannot load a model from app/models/spree/order_decorator.rb
Unable to annotate app/models/spree/page_decorator.rb: cannot load a model from app/models/spree/page_decorator.rb
Unable to annotate app/models/spree/payment_method_decorator.rb: cannot load a model from app/models/spree/payment_method_decorator.rb
Unable to annotate app/models/spree/product_decorator.rb: cannot load a model from app/models/spree/product_decorator.rb
Unable to annotate app/models/spree/products_container_decorator.rb: cannot load a model from app/models/spree/products_container_decorator.rb
Unable to annotate app/models/spree/taxon_decorator.rb: cannot load a model from app/models/spree/taxon_decorator.rb
Unable to annotate app/models/spree/variant_decorator.rb: cannot load a model from app/models/spree/variant_decorator.rb

That's pretty spammy...

@ctran
Copy link
Owner

ctran commented May 26, 2015

Are you suggesting that warnings are not displayed unless explicitly specified?

@alepore
Copy link
Author

alepore commented May 26, 2015

...or a way to disabled them.
i just want to point out that people may put different stuff under app/models and that message is not useful :)

@ctran ctran added the feature label May 26, 2015
@ctran ctran added this to the v2.6.11 milestone May 26, 2015
@ctran ctran self-assigned this May 26, 2015
@ctran ctran modified the milestones: v2.6.11, v2.7.0 Nov 18, 2015
@ctran ctran closed this as completed Dec 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants