standard-things / esm

Tomorrow's ECMAScript modules today!
Other
5.26k stars 146 forks source link

Entry.initialize(): detect properTime #832

Closed michaelfig closed 4 years ago

michaelfig commented 5 years ago

For npm packed modules, the mtime is ancient and so we use ctime instead

As discussed in https://github.com/standard-things/esm/issues/746#issuecomment-511160099

michaelfig commented 5 years ago

@jdalton Here's the PR!

jdalton commented 5 years ago

Thank you @michaelfig!

michaelfig commented 3 years ago

Thank you @michaelfig!

You're welcome!

@jdalton Can you please cut an NPM release with this change included? I notice that esm-3.2.25 doesn't have it.

michaelfig commented 3 years ago

@jdalton Can you please cut an NPM release with this change included? I notice that esm-3.2.25 doesn't have it.

I can do a forked NPM release (different package name) if you're too busy to do a proper one, but I'd prefer not to.