Skip to content

Commit

Permalink
Migrate to TypeScript (#32)
Browse files Browse the repository at this point in the history
* initial progress migrating to typescript

* initial progress migrating to typescript

* docs: update return types

* infra: remove sourceMaps from built files

* docs: update badges
  • Loading branch information
mrodrig committed Apr 11, 2023
1 parent ec6c049 commit a4b8f56
Show file tree
Hide file tree
Showing 23 changed files with 4,700 additions and 2,279 deletions.
1 change: 0 additions & 1 deletion .coveralls.yml

This file was deleted.

125 changes: 0 additions & 125 deletions .eslintrc

This file was deleted.

52 changes: 52 additions & 0 deletions .eslintrc.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,52 @@
module.exports = {
root: true,
env: {
es2022: true,
node: true,
mocha: true,
},
parserOptions: {
ecmaVersion: 13,
project: ['tsconfig.json'],
sourceType: 'module',
tsconfigRootDir: __dirname,
},
extends: [
'eslint:recommended',
'plugin:import/errors',
'plugin:import/warnings',
'plugin:import/typescript',
'plugin:@typescript-eslint/recommended',
],
parser: '@typescript-eslint/parser',
ignorePatterns: [
'/lib/**/*', // Ignore built files.
],
plugins: [
'@typescript-eslint',
'import',
],
rules: {
// Basic ES6
indent: [
'error', 4, { // use 4 spaces for indents
'SwitchCase': 1, // indent case within switch
}
],
'linebreak-style': 0, // mixed environment let git config enforce line endings
quotes: ['error', 'single'],
semi: ['error', 'always'],
'no-var': 'error',
'no-console': 0, // allow use of console.log,
'arrow-body-style': [0, 'always'],
'max-len': 0,
'camelcase': 1,
'import/no-unresolved': [
'error', {
// https://github.com/firebase/firebase-admin-node/discussions/1359
ignore: ['^firebase-admin/.+'],
},
],
'@typescript-eslint/consistent-type-definitions': 'warn'
},
};
42 changes: 42 additions & 0 deletions .github/workflows/automated-tests-workflow.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
name: Automated Node Version Tests

on: [push, pull_request]

jobs:
test:
runs-on: ${{ matrix.os }}

strategy:
matrix:
os: [ubuntu-latest]
node: [16, 18, 19]

steps:
- uses: actions/checkout@v3
- name: Use Node ${{ matrix.node }}
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node }}
- name: Install NPM Modules
run: npm ci
- name: Lint
run: npm run lint
- name: Test
run: |
npm run coverage
- name: Coveralls Parallel
uses: coverallsapp/github-action@master
with:
github-token: ${{ secrets.github_token }}
flag-name: node-${{ runner.os }}-${{ matrix.node }}
parallel: true

coveralls-finished:
needs: test
runs-on: ubuntu-latest
steps:
- name: Coveralls Finished
uses: coverallsapp/github-action@master
with:
github-token: ${{ secrets.github_token }}
parallel-finished: true
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,6 @@ node_modules

# Debug logs
npm-debug.log

# Built JavaScript files
lib
37 changes: 28 additions & 9 deletions .npmignore
Original file line number Diff line number Diff line change
@@ -1,11 +1,30 @@
test/
.git*
# Debug Logs
npm-debug.log
.travis.yml
.eslintrc
node_modules
coverage
upgrade_guides
_config.yml
.coveralls.yml

# Uncompiled Source Files
src
test

# Node Modules
node_modules/
node_modules/*

# Configuration Files
.eslintrc*
tsconfig*

# Test/Coverage Files
coverage/
coverage/*
.nycrc
.nyc_output
.nyc_output/*
.coveralls.yml
lib/test

# Github Configuration Files
_config.yml
.git*

# Extra documentation files
upgrade_guides/*
16 changes: 16 additions & 0 deletions .nycrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
{
"check-coverage": true,
"all": true,
"include": [
"src/**/!(*.test.*).[tj]s?(x)"
],
"exclude": [
"test/**/*.*"
],
"reporter": [
"lcov",
"text-summary",
"text"
],
"report-dir": "coverage"
}
20 changes: 0 additions & 20 deletions .travis.yml

This file was deleted.

0 comments on commit a4b8f56

Please sign in to comment.