It's not generally considered a good idea to have a data migration inside of a structural migration. I usually create a separate rake task for this sort of thing and then make a note that when I launch it, the second task has to be run... but that's not ideal either.

In this case I think it's fine to just leave it -- if it becomes a problem later we can consolidate migrations.