Skip to content

Commit

Permalink
Replace gulp and tslint (#298)
Browse files Browse the repository at this point in the history
* Replace gulp

* Combine test and coverage steps.

* Only build master branches.
(Build already happens for PR)

* Timeout and failure improvements.

* Improve PDF parsing and testing.

* Fix codecov warning.

* Make config files JSON.

* Cleanup.

* Fix typo [skip ci]
  • Loading branch information
westy92 authored May 10, 2021
1 parent 4a7eed4 commit 93bd4d7
Show file tree
Hide file tree
Showing 21 changed files with 2,055 additions and 4,122 deletions.
4 changes: 4 additions & 0 deletions .appveyor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@ skip_tags: true

clone_depth: 5

branches:
only:
- master

environment:
matrix:
- nodejs_version: 16
Expand Down
12 changes: 12 additions & 0 deletions .eslintrc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
{
"root": true,
"parser": "@typescript-eslint/parser",
"ignorePatterns": ["*.d.ts"],
"plugins": [
"@typescript-eslint"
],
"extends": [
"eslint:recommended",
"plugin:@typescript-eslint/recommended"
]
}
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jspm_packages
.node_repl_history

# Transpiled files
lib
dist

# phpStorm files
.idea
7 changes: 0 additions & 7 deletions .istanbul.yml

This file was deleted.

6 changes: 6 additions & 0 deletions .mocharc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
{
"extension": ["ts"],
"require": "ts-node/register",
"timeout": "15000",
"retries": 3
}
15 changes: 9 additions & 6 deletions .npmignore
Original file line number Diff line number Diff line change
@@ -1,9 +1,12 @@
coverage/
gulpfile.js
lib/test/
test/
tsconfig.json
tslint.json
.appveyor.yml
.eslintrc.json
.istanbul.yml
.mocharc.json
.nyc_output/
.nycrc
.travis.yml
.vscode/
coverage/
dist/test/
test/
tsconfig.json
3 changes: 3 additions & 0 deletions .nycrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
{
"reporter": ["text", "html", "json"]
}
6 changes: 5 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@ services:
git:
depth: 5

branches:
only:
- master

addons:
chrome: stable
before_install:
Expand All @@ -25,7 +29,7 @@ after_script:
# - greenkeeper-lockfile-upload

after_success:
- codecov -f coverage/coverage-remapped.json
- codecov --disable=gcov -f coverage/coverage-final.json

language: node_js
node_js:
Expand Down
16 changes: 4 additions & 12 deletions .vscode/launch.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,20 +5,12 @@
"version": "0.2.0",
"configurations": [
{
"name": "Unit Tests",
"type": "node",
"request": "launch",
"name": "Mocha Tests",
"program": "${workspaceRoot}/node_modules/mocha/bin/_mocha",
"args": [
"--timeout",
"15000",
"--colors",
"lib/test/**/*.js"
],
"outFiles": ["${workspaceRoot}/lib/**/*.js"],
"internalConsoleOptions": "openOnSessionStart",
"preLaunchTask": "default-build",
"smartStep": true
"runtimeExecutable": "npm",
"runtimeArgs": ["run", "test"],
"cwd": "${workspaceRoot}"
},
{
"type": "node",
Expand Down
7 changes: 5 additions & 2 deletions .vscode/tasks.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,11 @@
"version": "2.0.0",
"tasks": [
{
"type": "gulp",
"task": "scripts",
"type": "npm",
"script": "build",
"problemMatcher": [
"$tsc"
],
"label": "default-build",
"group": {
"kind": "build",
Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@ HTML to PDF converter via Chrome/Chromium.

## Prerequisites

* Latest Chrome/Chromium (latest recommended, 61 or higher required but some features may not work)
* Latest Chrome/Chromium
* Windows, macOS, or Linux
* Node.js 6 or later (we only test on 10+, mileage may vary)
* A [currently supported version of Node.js](https://nodejs.org/en/about/releases/)

## Installation

Expand Down
67 changes: 0 additions & 67 deletions gulpfile.js

This file was deleted.

Loading

0 comments on commit 93bd4d7

Please sign in to comment.