Skip to content

fix(turf-random): BREAKING: standardize randomPolygon and randomLineString options to camelCase - #3154

Merged
mfedderly merged 2 commits into
Turfjs:masterfrom
faizanjafar:fix/random-camelcase-3110
Aug 17, 2026
Merged

fix(turf-random): BREAKING: standardize randomPolygon and randomLineString options to camelCase#3154
mfedderly merged 2 commits into
Turfjs:masterfrom
faizanjafar:fix/random-camelcase-3110

Conversation

@faizanjafar

@faizanjafar faizanjafar commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Renames num_vertices -> numVertices, max_radial_length -> maxRadialLength in randomPolygon, and num_vertices -> numVertices, max_length -> maxLength, max_rotation -> maxRotation in randomLineString, to match the camelCase convention used across the rest of the library.

Fixes #3110

Please provide the following when creating a PR:

  • Meaningful title, including the name of the package being modified.
  • Summary of the changes.
  • Heads up if this is a breaking change.
  • Any issues this resolves.
  • Inclusion of your details in the contributors field of package.json - you've earned it! 👏
  • Confirmation that you've read the steps for preparing a pull request.

…ons to camelCase

Renames num_vertices -> numVertices, max_radial_length -> maxRadialLength
in randomPolygon, and num_vertices -> numVertices, max_length -> maxLength,
max_rotation -> maxRotation in randomLineString, to match camelCase
convention used across the rest of the library.(Turfjs#3110)
@mfedderly mfedderly added this to the v8 milestone Aug 17, 2026
@mfedderly mfedderly changed the title fix(turf-random): standardize randomPolygon and randomLineString options to camelCase fix(turf-random): BREAKING: standardize randomPolygon and randomLineString options to camelCase Aug 17, 2026

@mfedderly mfedderly left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you!

@mfedderly
mfedderly merged commit 4091360 into Turfjs:master Aug 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Standardize randomPolygon and randomLineString options from snake_case to camelCase

2 participants