On Wednesday, October 11, 2017 at 9:15:25 PM UTC-7, Tim Uckun wrote: > > Doing Sequel::Annotate.new(ModelClass).annotate(path) worked but it's > going to be cumbersome for the long run. > > Right now I have the following rake task > > desc "Update model annotations" > task :annotate do > require 'sequel/annotate' > Sequel::Annotate.annotate(Dir['app/models/*.rb']) > end > > It would be nice to be able to continue using task rather than annotating > them one by one when they fail. > > What's your thinking on why it doesn't recognize some classes but > recognizes others. For example one class it doesn't recognize is this > > > class HashedPassword < Sequel::Model > end >
You have two spaces between the class keyword and the class name, and the regexp used to match only allows a single space. That should be a simple fix if you want to simple a pull request. Thanks, Jeremy -- You received this message because you are subscribed to the Google Groups "sequel-talk" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/sequel-talk. For more options, visit https://groups.google.com/d/optout.
