When getting the last PR to work I saw significant opportunities to simplify. Some comments inline.
The most significant change is the removal of the CondaBootstrapEnv provider and requirements classes. There was too much copy and paste code in there and we're better served with a simple "bootstrap" flag in these classes. Heck I don't even know if we ought to even have the whole plugin/bootstrap support at this point, but for now, it is what it is.
mcg1969
commented
3 years ago
When getting the last PR to work I saw significant opportunities to simplify. Some comments inline.
The most significant change is the removal of the CondaBootstrapEnv provider and requirements classes. There was too much copy and paste code in there and we're better served with a simple "bootstrap" flag in these classes. Heck I don't even know if we ought to even have the whole plugin/bootstrap support at this point, but for now, it is what it is.