Migration files are only run once, you do not need to use effectively_installed() unless the changes may have been performed already without the migration file being run (besides during board installation).