Marc-André Lafortune

Results 450 comments of Marc-André Lafortune

Hi! Thanks for the pull request. I'm not against passing this info up, but I was wondering what was your use case for this?

So sorry this fell through the cracks. Looks like a good idea, although we might as well post the initial as well as final touch event, what do you think?

No initializer is needed. I believe you're looking for `calendar_date_select_tag` (i.e. missing the `_tag` part). I think `calendar_date_select` is available only from a form (e.g. `form_for blabla do |f| ;...

Cool. As for (1), you could embed it by default only if "emebedded_interpreter" is true. Otherwise, documentation for (2) in the installation notes would do too. Thanks!

I coded up some parsing methods that apply for any web site; the methods are in a module which I include in Nokogiri document. Later on I needed some custom...

I'll be a contrarian (again) and say that introducing `delete_by` and `destroy_by` was a mistake. There is a cost of introducing a method (besides added maintenance): it augments *cognitive load*....

(I would even approve the contrary rule: do not use `delete_by`/`destroy_by` as they are obscure, error prone and confusing, but I'm not expecting everyone would agree with this)

Anyone interested can refer to the original issue: https://github.com/rails/rails/issues/35304 (there wasn't much support either)

Oops, fixed the link to the [1m37](https://github.com/rubocop/rubocop-ast/runs/2062618835?check_suite_focus=true) one. And no link for 10min run, as I did "restart all jobs" and that seem to delete the log. There was just...