@influxdata/giraffe - npm Package Compare versions

Comparing version 0.16.10 to 0.16.11

package.json

 {
   "name": "@influxdata/giraffe",
-  "version": "0.16.10",
+  "version": "0.16.11",
   "main": "dist/index.js",
@@ -28,4 +28,4 @@ "license": "MIT",
     "@types/react-dom": "^16.8.2",
-    "@typescript-eslint/eslint-plugin": "^1.5.0",
-    "@typescript-eslint/parser": "^1.5.0",
+    "@typescript-eslint/eslint-plugin": "^2.10.0",
+    "@typescript-eslint/parser": "^2.10.0",
     "d3-array": "^2.0.3",
@@ -38,8 +38,8 @@ "d3-color": "^1.2.3",
     "d3-shape": "^1.3.5",
-    "eslint": "^5.15.3",
-    "eslint-config-prettier": "^5.0.0",
-    "eslint-plugin-prettier": "^3.0.1",
-    "eslint-plugin-react": "^7.12.4",
+    "eslint": "^6.7.2",
+    "eslint-config-prettier": "^6.7.0",
+    "eslint-plugin-prettier": "^3.1.1",
+    "eslint-plugin-react": "^7.17.0",
     "intl-dateformat": "^0.1.1",
-    "jest": "^24.5.0",
+    "jest": "^24.9.0",
     "memoize-one": "^5.0.2",
@@ -50,9 +50,9 @@ "prettier": "^1.16.4",
     "react-virtualized-auto-sizer": "^1.0.2",
-    "rollup": "^1.10.0",
-    "rollup-plugin-commonjs": "^10.0.0",
+    "rollup": "^1.27.8",
+    "rollup-plugin-commonjs": "^10.1.0",
     "rollup-plugin-gzip": "^2.2.0",
-    "rollup-plugin-node-resolve": "^5.0.3",
+    "rollup-plugin-node-resolve": "^5.2.0",
     "rollup-plugin-sourcemaps": "^0.4.2",
-    "rollup-plugin-terser": "^5.0.0",
-    "rollup-plugin-typescript2": "^0.21.2",
+    "rollup-plugin-terser": "^5.1.2",
+    "rollup-plugin-typescript2": "^0.25.3",
     "ts-jest": "^24.0.0",
@@ -59,0 +59,0 @@ "typescript": "^3.3.4000",

src/transforms/histogram.ts

@@ -170,3 +170,3 @@ import {range, thresholdSturges} from 'd3-array'
 
-  let binTable = newTable(binCount * groupIDs.length)
+  const binTable = newTable(binCount * groupIDs.length)
     .addColumn(X_MIN, xColType, xMinData)
@@ -173,0 +173,0 @@ .addColumn(X_MAX, xColType, xMaxData)

src/utils/drawPoints.ts

@@ -40,3 +40,3 @@ import {Scale, NumericColumnData, SymbolType} from '../types'
 
-  for (var i = 0; i < n; i++) {
+  for (let i = 0; i < n; i++) {
     const rowIndex = rowIndices ? rowIndices[i] : i
@@ -43,0 +43,0 @@ const x = xScale(xColData[rowIndex])

src/utils/formatters.test.ts

@@ -18,5 +18,35 @@ import {
     // Defaults to 12 hour time when not UTC
-    expect(nonUTCFormatter(d)).toEqual('2018-12-31 4:00:00 PM PST')
+    expect(nonUTCFormatter(d)).toEqual('2018-12-31  4:00:00 PM PST')
   })
 
+  test('uses AM/PM when given "a" in the format regardless of time zone or time format', () => {
+    const utcFormatterWithFormat = timeFormatter({
+      timeZone: 'UTC',
+      format: 'YYYY-MM-DD HH:mm:ss a ZZ',
+    })
+    const utcFormatterWithFormatWithLowerH = timeFormatter({
+      timeZone: 'UTC',
+      format: 'YYYY-MM-DD hh:mm:ss a ZZ',
+    })
+    const nonUTCFormatterWithFormat = timeFormatter({
+      timeZone: 'America/Los_Angeles',
+      format: 'YYYY-MM-DD HH:mm:ss a ZZ',
+    })
+    const nonUTCFormatterWithFormatWithLowerH = timeFormatter({
+      timeZone: 'America/Los_Angeles',
+      format: 'YYYY-MM-DD hh:mm:ss a ZZ',
+    })
+
+    const d = new Date('2019-01-01T00:00Z')
+
+    expect(utcFormatterWithFormat(d)).toEqual('2019-01-01 12:00:00 AM UTC')
+    expect(utcFormatterWithFormatWithLowerH(d)).toEqual(
+      '2019-01-01 12:00:00 AM UTC'
+    )
+    expect(nonUTCFormatterWithFormat(d)).toEqual('2018-12-31 4:00:00 PM PST')
+    expect(nonUTCFormatterWithFormatWithLowerH(d)).toEqual(
+      '2018-12-31  4:00:00 PM PST'
+    )
+  })
+
   test('can format times with format strings', () => {
@@ -29,2 +59,3 @@ const tests = [
       ['YYYY/MM/DD HH:mm:ss', '2019/01/01 00:00:00'],
+      ['hh:mm a', '12:00 AM'],
       ['HH:mm a', '12:00 AM'],
@@ -31,0 +62,0 @@ ['HH:mm', '00:00'],

src/utils/formatters.ts

@@ -142,2 +142,5 @@ /*
   const formatStringFormatter = createDateFormatter({
+    // a deliberate space in front of single digit hours keeps the tick label length
+    // and the total number of ticks consistent regardless of time frame
+    hh: ({hour}) => (Number(hour) < 10 ? ` ${Number(hour)}` : String(hour)),
     HH: ({lhour}) => {
@@ -144,0 +147,0 @@ if (format && format.includes('a')) {

src/utils/simplify.ts

@@ -63,3 +63,3 @@ type ArrayLike = number[] | Float64Array
   if (dx !== 0 || dy !== 0) {
-    var t = ((x2 - x) * dx + (y2 - y) * dy) / (dx * dx + dy * dy)
+    const t = ((x2 - x) * dx + (y2 - y) * dy) / (dx * dx + dy * dy)
 
@@ -66,0 +66,0 @@ if (t > 1) {

src/utils/useDragEvent.ts

@@ -41,3 +41,4 @@ import * as React from 'react'
 
-        let {direction, initialX, initialY} = dragEventRef.current
+        const {initialX, initialY} = dragEventRef.current
+        let {direction} = dragEventRef.current
 
@@ -44,0 +45,0 @@ if (!direction) {

src/utils/useHoverPointIndices.ts

@@ -293,3 +293,3 @@ import {range} from 'd3-array'
 const collectClosestRowIndices = (i0, i1, j0, j1, bins): number[] => {
-  let closestRowIndices = []
+  const closestRowIndices = []
 
@@ -372,3 +372,3 @@ let x = i0
 
-  let nearestIndexByGroup: NearestIndexByGroup = {}
+  const nearestIndexByGroup: NearestIndexByGroup = {}
 
@@ -375,0 +375,0 @@ collectNearestIndices(

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Uses eval

Supply chain risk

Package uses dynamic code execution (e.g., eval()), which is a dangerous practice. This can prevent the code from running in certain environments and increases the risk that the code may contain exploits or malicious behavior.

Found 1 instance in 1 package

Mixed license

License

(Experimental) Package contains multiple licenses.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Uses eval

Supply chain risk

Package uses dynamic code execution (e.g., eval()), which is a dangerous practice. This can prevent the code from running in certain environments and increases the risk that the code may contain exploits or malicious behavior.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

1498642

increased by29.91%

Number of package files

566

increased by60.8%

Lines of code

7074

increased by0.64%

Worsened metrics

Number of low license alerts

1

increased byInfinity%

No dependency changes