You’ve signed the CLA, vinothkannans. Thank you! This pull request is ready for review.
# ignore auto-generated plugin js assets
Using equality (
==) is slightly easier to read than inequality (
next if args[:include_disabled] == false && !plugin.enabled?
I tend to prefer
find_all because it’s shorter and I don’t confuse it with
plugins.select do |plugin|
I spent many hours to keep existing or create new spec tests for this PR. But it looks it’s not possible since all the dynamic file creations are happening only on initial app booting time.