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

Feature/#354 deprecation of jql search and evaluate expression endpoints #355

Conversation

nessgor
Copy link

@nessgor nessgor commented Feb 6, 2025

Fixes #354.

`npm fix audit`
RangeError: options.minThreads and options.maxThreads must not conflict

Fixed by adding `--minWorkers=8` to `test:unit` in package.json
Only implemented unit test for the newly added function evaluateJiraExpressionUsingEnhancedSearch()
Implemented new function evaluateJiraExpressionUsingEnhancedSearch()
Only implemented unit test for the newly added function evaluateJiraExpressionUsingEnhancedSearch()
Implemented new function evaluateJiraExpressionUsingEnhancedSearch()
@MrRefactoring MrRefactoring self-requested a review February 6, 2025 23:13
@MrRefactoring MrRefactoring added this to the v4.1.0 milestone Feb 7, 2025
@MrRefactoring MrRefactoring changed the base branch from master to release/v4.1.0 February 7, 2025 22:38
package.json Outdated Show resolved Hide resolved
@MrRefactoring MrRefactoring added the enhancement Denotes a suggestion or request aimed at improving or adding new features to the project. label Feb 7, 2025
@MrRefactoring
Copy link
Owner

To be honest, I was working on these same improvements in parallel, but not all the naming was done as well as you’ve done it, so I suggest merging release/v4.1.0 changes into the your branch branch and adjusting the naming where you see fit.

@nessgor
Copy link
Author

nessgor commented Feb 8, 2025

Hi @MrRefactoring, I don't have a strong opinion on the naming. I just copied them from the Jira API documentation and got some inspiration from your existing code. Could you suggest some code or group that you would like me to review for the naming? Otherwise, I think it's fine 👍

@MrRefactoring
Copy link
Owner

You can start with the methods you’ve already added. After merging with the release branch, the names might get overwritten by my changes, and I’d like to preserve yours

…ion-of-JQL-search-and-Evaluate-expression-endpoints
@MrRefactoring MrRefactoring merged commit affed8e into MrRefactoring:release/v4.1.0 Feb 8, 2025
@nessgor nessgor deleted the feature/#354-Deprecation-of-JQL-search-and-Evaluate-expression-endpoints branch February 9, 2025 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Denotes a suggestion or request aimed at improving or adding new features to the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecation of JQL search and Evaluate expression endpoints
2 participants