proposal: stabilize `std/internal`

denoland / std

The Deno Standard Library

https://jsr.io/@std

MIT License

3.08k stars 609 forks source link

proposal: stabilize `std/internal` #4739

Closed iuioiua closed 5 months ago

iuioiua commented 5 months ago

Requirements:

[x] 100% documented, passing deno doc --lint checks and adhering to the documentation guidelines.
[x] Maximum possible test coverage

This is to unblock dependents from being stabilized, so we're skipping 1.0.0-rc.1 and going straight to 1.0.0.

Towards #4600

timreichen commented 5 months ago

Std/internal has diff() and diffstr(). Is there a reason why they are abbreviated and not written out?

iuioiua commented 5 months ago

What a "diff" is is common knowledge. I don't think anyone's had any issues understanding it, nor do I think anyone will. If we do, we can reconsider, but it's fine.

timreichen commented 5 months ago

What a "diff" is is common knowledge. I don't think anyone's had any issues understanding it, nor do I think anyone will. If we do, we can reconsider, but it's fine.

Ok, but there are datetime/difference.ts and semver/difference.ts which are written out, and commonly WebApis do not use abbreviations (Set.prototype.difference(), Set.prototype.symmetricDifference()). Just thinking about consistency across multiple modules and apis.