eslint-plugin-neverthrow
eslint-plugin-neverthrow copied to clipboard
mdbetancourt
I can create a PR to help solve #9 and #10. I need a confirm that this repository is still worth updating. I hope anyone is still maintaining this.
I just notice that the following case is not checked by the linter: ```ts function foo(): void { const arr: Result[] = []; const resultFromArray = arr[0]!; // should raise...
Fix: #10
**Changes** 1. Refactor and fix: Wrap the assignment check using a helper `handleAssignation`. 2. Feat: Adding `AwaitExpression` node selector (this fix #10) 3. Feat: Try to mark the first assigned...
## :rotating_light: The automated release from the `master` branch failed. :rotating_light: I recommend you give this issue a high priority, so other packages depending on you can benefit from your...
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "github-actions[bot] avatar"){kind=avatar}
Shouldn't it be allowed to handle errors with an if/else block and the isOk/isErr functions? E.g. these should now throw an error: ``` const result = getResult(); if (result.isOk()) {...
After a major update of @typescript-eslint/parser there are some problems parsing types from neverthrow. While linting I've got the following: ``` ESLint: 8.45.0 Error: Error while loading rule 'neverthrow/must-use-result': types...
If the result is returned in a Promise, this does never seem to trigger an error: `const result = await getResult();`
In my opinion it should be allowed to save the result for later use, or call a method with it. At the moment both use-cases can trigger the linter rule....
# Work In Progress It turns out that `combine` is currently not being handled. This PR (currently **work in progress**) implements the ability for `eslint-plugin-neverthrow` to omit lint errors when...
