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

Add the processing error #99

Merged
merged 1 commit into from
Nov 27, 2024
Merged

Add the processing error #99

merged 1 commit into from
Nov 27, 2024

Conversation

w3nl
Copy link
Contributor

@w3nl w3nl commented Nov 27, 2024

No description provided.

@w3nl w3nl requested a review from Copilot November 27, 2024 10:07
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 6 changed files in this pull request and generated no suggestions.

Files not reviewed (1)
  • package.json: Language not supported
Comments skipped due to low confidence (2)

src/processing-error.js:18

  • The method name 'errorStatus' should be renamed to 'status' to maintain consistency with other error classes.
get errorStatus() {

src/tests/processing-error.unit.js:37

  • The expected error message 'Invalid error' is not defined in the ProcessingError class. This might be a bug.
assert.deepEqual(error.message, 'Invalid error')

@w3nl w3nl merged commit c439d86 into main Nov 27, 2024
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants