Releases · approvals/Approvals.NodeJS

Normally I would set this as a patch release, but I making it a major version as I am concerned about changes to have javascript and typescript handle exports.

I have had to use the following a bit

Imports

var a = require("a")
⇓
import {a} from "a"
or 
import * as a from "a"

Assets 2

31 Oct 17:35

claremacrae

v6.2.4

4811641

Fix JestReporter importing full class instead of type

For the future, we have set "importsNotUsedAsValues" to "error" to prevent future recurrences.

This helps other packages that have this value set not error-out because they are using Approvals.

Assets 2

31 Oct 16:40

isidore

v6.2.3

b0e01eb

fix version number in npm package

Fixes #150
the 'v' in v6.2.3 caused trouble with npm

Assets 2

08 Sep 23:12

isidore

v6.2.2

9e722f9

Minor bug with Angular + Jest

Added check when running JestApprovals in angular

Assets 2

10 Apr 21:07

claremacrae

v6.2.1

e642564

v6.2.1: Fix some warnings about imports

This fixes the following warnings in v6.2.0

$ eslint ./src --fix && eslint ./tests --fix && tsc --noEmit --pretty && svelte-check
node_modules/approvals/lib/Core/Options.ts:2:1 - error TS1371: This import is never used as a value and must use 'import type' because 'importsNotUsedAsValues' is set to 'error'.
2 import {Namer} from "../Namer";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
node_modules/approvals/lib/Providers/Jest/JestApprovals.ts:3:1 - error TS6133: 'getJestNamer' is declared but its value is never read.
3 import {getJestNamer} from "./JestNamer";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Found 2 errors in 2 files.
Errors  Files
     1  node_modules/approvals/lib/Core/Options.ts:2
     1  node_modules/approvals/lib/Providers/Jest/JestApprovals.ts:3

Assets 2