No thats not a bad practice - its pretty common to use the inherit_from feature for many configurations for the same plugin. Also editing the meltano.yml manually is also pretty common, you just have to be more careful that its valid and now after the v3.0 that you've locked your plugin definitions. You can run
meltano compile
to make sure everything is valid