search

UrbanInstitute / education-data-package-stata

MIT License
19 stars 4 forks source link

Erases data in memory #11

Closed grahamimac closed 6 years ago

grahamimac commented 6 years ago

From Matt: Don’t let users run the package if they have data in memory, as they will lose it without warning. Instead, I suggest giving them an error message, so they have to type “clear” before running the package (or this could be an option, but default should not erase their data without warning them)

grahamimac commented 6 years ago

I have replaced the "clear" function with a check to see if a dataset exists, and then if one exists, aborting the command and returning a message stating they need to clear and re-run to run the package. This will be resolved in the next release.

ericatheresa commented 6 years ago

@grahamimac Is it possible to just add the option "clear" so that you can do it in one step? (this is how stata commands generally work, ie "use datasetname, clear") Not high priority, obviously.

grahamimac commented 6 years ago

This should be added in the next release on Friday, great idea.