refactor: default annotate_rendered_view_with_filenames to true in development

[adrianthedev]

Motivation / Background

This Pull Request has been created to set config.action_view.annotate_rendered_view_with_filenames to true in development.
This is such a powerful tool, and I find myself (and probably others too) turning it on for every project in the development environment.

This also disables the feature in tests, so users can test views without having the comments muddy up the output.

Detail

This Pull Request changes development.rb.tt and un-comments the line that sets this setting to true.

Additional information

Checklist

Before submitting the PR make sure the following are checked:

• This Pull Request is related to one change. Changes that are unrelated should be opened in separate PRs.
• Commit message has a detailed description of what changed and why. If this PR fixes a related issue include it in the commit message. Ex: [Fix #issue-number]
• Tests are added or updated if you fix a bug or add a feature. Not needed.
• CHANGELOG files are updated for the changed libraries if there is a behavior change or additional feature. Minor bug fixes and documentation changes should not be included.

[adrianthedev]

It's my first PR to Rails, and I'm not sure why the option is applied in the test environment.
Can anyone point me in the right direction?

Ideally, the option would be turned off for all tests so we can test without the comments.

[adrianthedev]

I'm following the code and see that in build_app the environment is set to development.
Probably to test it as close as possible to real-world.

[rafaelfranca]

Can you please fix the build?

[adrianthedev]

Yup. On it!

[adrianthedev]

Thanks @rafaelfranca!