Skip to content
New issue

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

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

Already on GitHub? # to your account

Support trailing comma after the last argument. #520

Merged
merged 1 commit into from
May 3, 2024

Conversation

linuxerwang
Copy link
Contributor

Example:

function mergeObjects(x, y) { /* dummy body */}

mergeObjects(
    { "duck": "quack" },
    { "dog": "bark" }, // Allow trailing comma after the last argument.
);

Example:

```js
function mergeObjects(x, y) { /* dummy body */}

mergeObjects(
    { "duck": "quack" },
    { "dog": "bark" }, // Allow trailing comma after the last argument.
);
```
@stevenh stevenh merged commit 2d23528 into robertkrimen:master May 3, 2024
2 checks passed
@tlimoncelli
Copy link

Thank you so much! I've wanted this feature for years!

(Take a peek at https://github.com/StackExchange/dnscontrol/releases/tag/v4.15.0)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants