Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deprecation(semver): rename rangeFormat() to formatRange() #4090

Merged
merged 13 commits into from
Jan 8, 2024

Conversation

timreichen
Copy link
Contributor

  • renames rangeFormat() to formatRange()

@timreichen timreichen requested a review from kt3k as a code owner January 4, 2024 10:53
@github-actions github-actions bot added the semver label Jan 4, 2024
@timreichen timreichen mentioned this pull request Jan 4, 2024
13 tasks
semver/range_test.ts Outdated Show resolved Hide resolved
@kt3k
Copy link
Member

kt3k commented Jan 4, 2024

I don't see the reason of the deprecation. Is this discussed somewhere?

@timreichen
Copy link
Contributor Author

timreichen commented Jan 4, 2024

I don't see the reason of the deprecation. Is this discussed somewhere?

Javascript generally has a verb-first convention for functions. It is highlighted in the #3948 (comment) description.

semver/range_format.ts Outdated Show resolved Hide resolved
@kt3k
Copy link
Member

kt3k commented Jan 5, 2024

Javascript generally has a verb-first convention for functions. It is highlighted in the #3948 (comment) description.

Fair enough.

Copy link
Contributor

@iuioiua iuioiua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added a format_range_test.ts file. Now, LGMT. Thank you.

@iuioiua iuioiua requested a review from kt3k January 6, 2024 22:32
Copy link
Member

@kt3k kt3k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@kt3k kt3k merged commit 3d74dbd into denoland:main Jan 8, 2024
12 checks passed
@timreichen timreichen deleted the semver_rename_range_format branch January 8, 2024 09:20
timreichen added a commit to timreichen/std that referenced this pull request Jan 8, 2024
…and#4090)

Co-authored-by: Asher Gomez <ashersaupingomez@gmail.com>
iuioiua added a commit that referenced this pull request Jan 11, 2024
* initial commit

* Update log/logger.ts

Co-authored-by: Asher Gomez <ashersaupingomez@gmail.com>

* replace WARNING with WARN

* update

* refactor(cli): make `Spinner` constructor options more readable (#4091)

* initial commit

* update

* docs(testing): fix assertion deprecation notices (#4122)

* refactor(assert): remove `void` return types (#4124)

* fix(http): punt removal version of `unstable_cookie_map.ts` (#4125)

* chore: ignore `/docs` in license checker tool (#4123)

* deprecation(semver): rename `rangeFormat()` to `formatRange()` (#4090)

Co-authored-by: Asher Gomez <ashersaupingomez@gmail.com>

* refactor(archive): prepare for `noUncheckedIndexedAccess` (#4110)

* refactor(archive): prepare for noUncheckedIndexedAccess

* adjust test to handle new TarHeader requirements

* revert WARNING

---------

Co-authored-by: Asher Gomez <ashersaupingomez@gmail.com>
Co-authored-by: Simon Holloway <simon@syhol.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants