Skip to content

Commit

Permalink
Merge branch 'master' into cicd/add-bug-and-community-ideas-form
Browse files Browse the repository at this point in the history
  • Loading branch information
rsaz authored Oct 22, 2024
2 parents 70fc24d + 6e97383 commit 16c6d8f
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 6 deletions.
7 changes: 5 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,10 @@ jobs:
- name: Run tests
run: npm test --coverage
- name: Store code coverage report
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: coverage
overwrite: true
path: coverage/

Build:
Expand Down Expand Up @@ -61,6 +62,7 @@ jobs:

Upload_Coverage_Report:
name: Upload coverage report to codecov
environment: CI
needs: [Testing]
runs-on: ubuntu-latest
steps:
Expand All @@ -74,7 +76,8 @@ jobs:
name: coverage
path: coverage/
- name: Codecov Upload
uses: codecov/codecov-action@v3
uses: codecov/codecov-action@v4
with:
directory: coverage/
fail_ci_if_error: true
token: ${{ secrets.CODECOV_TOKEN }}
4 changes: 2 additions & 2 deletions src/planning/planner.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import * as ERROR_MSGS from '../constants/error_msgs';
import { BindingTypeEnum, TargetTypeEnum } from '../constants/literal_types';
import * as METADATA_KEY from '../constants/metadata_keys';
import { interfaces } from '../interfaces/interfaces';
import { isStackOverflowExeption } from '../utils/exceptions';
import { isStackOverflowException } from '../utils/exceptions';
import { circularDependencyToException, getServiceIdentifierAsString, listMetadataForTarget, listRegisteredBindingsForServiceIdentifier } from '../utils/serialization';
import { Context } from './context';
import { Metadata } from './metadata';
Expand Down Expand Up @@ -236,7 +236,7 @@ function plan(
return context;
} catch (error) {
if (
isStackOverflowExeption(error)
isStackOverflowException(error)
) {
circularDependencyToException(context.plan.rootRequest);
}
Expand Down
4 changes: 2 additions & 2 deletions src/utils/exceptions.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import * as ERROR_MSGS from '../constants/error_msgs';

export function isStackOverflowExeption(error: unknown): error is RangeError {
export function isStackOverflowException(error: unknown): error is RangeError {
return (
error instanceof RangeError ||
(error as Error).message === ERROR_MSGS.STACK_OVERFLOW
Expand All @@ -11,7 +11,7 @@ export const tryAndThrowErrorIfStackOverflow = <T>(fn: () => T, errorCallback: (
try {
return fn();
} catch (error) {
if (isStackOverflowExeption(error)) {
if (isStackOverflowException(error)) {
error = errorCallback();
}
throw error;
Expand Down

0 comments on commit 16c6d8f

Please sign in to comment.