algolia / algolia/algolia-filters-js-syntax-validator

Suggestion: make Parser.parse static

Open
#1 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
JavaScript
Stars
4
Forks
2
PR merge metrics
No merged PRs in 30d

Description

https://github.com/algolia/algolia-filters-js-syntax-validator/blob/6446be2cc325c726cb1450b626a9d1acc5a930fb/src/parser.js#L4

If this would be instead `static parse(s)`, it would be able to be used as:

```js
const Parser = require('algolia-filters-js-syntax-validator');

const response = Parser.parse('A or B');
```

Not exactly necessary of course, but would be simpler IMO

Feel free to close if you think otherwise

Contributor guide

No contributing guide indexed for this repository

Research direction

Read src/parser.js at the linked line to understand the current Parser.parse entry point and how the module is exported. Verify that the requested static API supports Parser.parse('A or B') while preserving the existing parsing behavior; the issue does not mention a test file to run.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
tooling
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.