illinois-ceesd / emirge

Environment for MirgeCom
MIT License
4 stars 3 forks source link

rename loopy #85

Closed matthiasdiener closed 3 years ago

matthiasdiener commented 3 years ago

cc https://github.com/illinois-ceesd/mirgecom/pull/136

inducer commented 3 years ago

I saw this too, but:

So I think removing this is premature.

matthiasdiener commented 3 years ago

I saw this too, but:

  • It'll keep working in the presence of the rename.
  • Removing it now will break the workflow of folks who run a new emirge with an old mirgecom.

So I think removing this is premature.

Ok, but I'll document this at least.

inducer commented 3 years ago

Ok, but I'll document this at least.

Fair point.

matthiasdiener commented 3 years ago

ready for review.