JMSLab / xtevent

Stata package -xtevent-
MIT License
43 stars 12 forks source link

Check required Stata version #175

Closed jorpppp closed 4 months ago

jorpppp commented 4 months ago

Most of our programs have version 11.2 statements. Check if this is correct and update version statements accordingly.

jorpppp commented 4 months ago

We do require version 6.x of reghdfe when using reghdfe. From https://github.com/sergiocorreia/ivreghdfe/blob/cc35a985032931750036c5762cc88415f2f1c04c/src/ivreghdfe.ado#L52 reghdfe 6.x requires version 13, so it may make sense to update our version statements to version 13.

jorpppp commented 4 months ago

Note that eventstudyinteract from where we get our Sun and Abraham code also requires Stata 13: https://github.com/lsun20/EventStudyInteract/blob/87d16affd23f8bea20cb312c9e02fea0f1d87fe5/eventstudyinteract.ado#L4

jorpppp commented 4 months ago

Among other packages for diff in diff and event studies:

jorpppp commented 4 months ago

From the above I will update the version statements to version 13

jorpppp commented 4 months ago

Summary: In this issue we changed version statements from 11.2 to 13, to be consistent with the version required in package dependencies.