Marcelo

Results 22 comments of Marcelo

Yes, very handy.

Since there seems to be an agreement on simply adding a clarification for this guideline, I created a PR for it. I only modified the `read_attribute` part, not `write_attribute`. For...

I'd just point out something that wasn't immediately obvious to me, regarding when `ActiveModel::MissingAttributeError` is raised. ActiveRecord's documentation [has this example](https://api.rubyonrails.org/classes/ActiveRecord/AttributeMethods.html#method-i-5B-5D): ```ruby person = Person.new(name: 'Francesco', age: '22') person[:name] #...

Sorry for adding a PR that is not finished, but this way it is already clear where I'm going and I can get some feedback before adding more tests. I...

All flags that define the created docker service were tested with unit tests.

A binary release would be great.

@pirj that's a good point, I can adapt the PR to take this into account. How about this? > Prefer `self[:attribute]` over `read_attribute(:attribute)`, if the latter is used without a...

After looking more into the Rails code, I found out the conclusion from the referenced discussion is not exactly correct. If I just try any random attribute with `[]`, I...

> Did you put a breakpoint to somewhere you want to stop? Yes, and without `bundle exec` it works fine.