Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
54 commits
Select commit Hold shift + click to select a range
2b27c68
add exec_wrapper function
nickfyson Aug 21, 2020
45e00a8
match existing behaviour when custom listeners defined
nickfyson Aug 24, 2020
cd22abc
add example regex match for stdout/err string
nickfyson Aug 24, 2020
7b7e0e1
update error_wrapper to take matcher array
nickfyson Sep 1, 2020
6ef5334
wrap more codeql calls in the error catcher wrapper
nickfyson Sep 1, 2020
231537b
refactor to handle exec.exec return values together
nickfyson Sep 2, 2020
6a43a61
ensure matchers not applied if exit code is zero
nickfyson Sep 3, 2020
dd4e841
remove spurious use of exec_wrapper
nickfyson Sep 3, 2020
9701a93
ensure exec_wrapper always captures exit code of called process
nickfyson Sep 7, 2020
1f3ce75
add initial suite of tests for exec_wrapper
nickfyson Sep 7, 2020
7dbaff0
move error matcher definition into dedicated file
nickfyson Sep 7, 2020
3e6d239
add test for initial error matcher
nickfyson Sep 7, 2020
cc2dfaf
clean up and rename things to follow proper conventions
nickfyson Sep 7, 2020
30bb37a
restore all default workflow triggers and integration tests
nickfyson Sep 7, 2020
c4f579a
remove the intentionally failing matcher integration tests
nickfyson Sep 7, 2020
3cd4127
Merge branch 'main' into nickfyson/error_wrapper
nickfyson Sep 7, 2020
88951d6
renames to reflect the switch to using toolrunner
nickfyson Sep 7, 2020
0539269
split up builtin and custom queries
robertbrignull Sep 10, 2020
1fb7c81
tweak in response to reviewer comments
nickfyson Sep 11, 2020
b3b9901
make ErrorMatcher an object rather than a tuple
nickfyson Sep 11, 2020
b104d6e
Merge branch 'main' into nickfyson/error_wrapper
nickfyson Sep 11, 2020
814840b
Merge branch 'main' into nickfyson/error_wrapper
nickfyson Sep 14, 2020
06765f9
Adding ESLint config and required dev dependencies
Sep 14, 2020
09b4a82
Removing the tslint config
Sep 14, 2020
c96f843
Regenerating node_modules
Sep 14, 2020
a184d50
Running lint-fix
Sep 14, 2020
cb51c85
Adding temporary overrides to allow the linter to pass
Sep 14, 2020
7bdf90a
Adding super simple lint Action
Sep 14, 2020
698eab0
Merge pull request #181 from github/cbraynor/eslint
Sep 14, 2020
e5e9aad
Merge branch 'main' into nickfyson/error_wrapper
nickfyson Sep 14, 2020
245c02c
Merge pull request #174 from github/nickfyson/error_wrapper
nickfyson Sep 14, 2020
89dad14
Replace all occurrences
robertbrignull Sep 14, 2020
121fd33
Introduce actions-util.ts
robertbrignull Sep 15, 2020
1ca9d4c
Merge pull request #207 from github/robertbrignull/replace_all
robertbrignull Sep 15, 2020
3325da8
Remove unused files
marcogario Sep 15, 2020
0da3e94
Merge branch 'main' into remove_file
nickfyson Sep 15, 2020
4076e23
Merge pull request #209 from github/remove_file
marcogario Sep 15, 2020
d88fa5c
Add queries
robertbrignull Sep 15, 2020
c1cee53
Add getOptionalInput and getRequiredInput
robertbrignull Sep 15, 2020
7be1a41
Merge branch 'main' into robertbrignull/consistent_inputs
robertbrignull Sep 16, 2020
4c4114f
add an extra getRequiredInput
robertbrignull Sep 16, 2020
090a701
add explanation to query
robertbrignull Sep 16, 2020
ef3b75c
Update action.yml
robertbrignull Sep 15, 2020
5f59277
Merge branch 'main' into main-v1
marcogario Sep 16, 2020
2c94a7f
Merge pull request #215 from github/main-v1
robertbrignull Sep 17, 2020
4b4f9e8
Merge branch 'main' into robertbrignull/consistent_inputs
robertbrignull Sep 17, 2020
c9b0611
Merge pull request #211 from github/robertbrignull/consistent_inputs
robertbrignull Sep 17, 2020
ade519b
Reduce triggers in workflows
marcogario Sep 17, 2020
875a8da
include workflow queries in status report
robertbrignull Sep 17, 2020
5166e75
Merge pull request #218 from github/marcogario/reduce_ci_jobs
marcogario Sep 17, 2020
1dc1029
Merge branch 'main' into split_builtin_custom_queries
robertbrignull Sep 18, 2020
55458a1
Merge pull request #179 from github/split_builtin_custom_queries
robertbrignull Sep 18, 2020
def0091
Merge branch 'main' into robertbrignull/init_status_queries
robertbrignull Sep 18, 2020
b2dfa6e
Merge pull request #219 from github/robertbrignull/init_status_queries
robertbrignull Sep 18, 2020
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
The diff you're trying to view is too large. We only load the first 3000 changed files.
5 changes: 5 additions & 0 deletions .eslintignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
**/webpack.config.js
lib/**
runner/dist/**
src/testdata/**
tests/**
59 changes: 59 additions & 0 deletions .eslintrc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,59 @@

{
"parser": "@typescript-eslint/parser",
"parserOptions": {
"project": "./tsconfig.json"
},
"plugins": ["@typescript-eslint", "filenames", "github", "import", "no-async-foreach"],
"extends": [
"eslint:recommended",
"plugin:@typescript-eslint/recommended",
"plugin:@typescript-eslint/recommended-requiring-type-checking",
"plugin:github/recommended",
"plugin:github/typescript"
],
"rules": {
"filenames/match-regex": ["error", "^[a-z0-9-]+(\\.test)?$"],
"import/extensions": "error",
"import/no-amd": "error",
"import/no-commonjs": "error",
"import/no-dynamic-require": "error",
"import/no-extraneous-dependencies": ["error", {"devDependencies": false}],
"import/no-namespace": "off",
"import/no-unresolved": "error",
"import/no-webpack-loader-syntax": "error",
"no-async-foreach/no-async-foreach": "error",
"no-console": "off",
"no-sequences": "error",
"one-var": ["error", "never"],
"sort-imports": ["error", { "allowSeparatedGroups": true }]
},
"overrides": [{
// "temporarily downgraded during transition to eslint
"files": "**",
"rules": {
"@typescript-eslint/ban-types": "off",
"@typescript-eslint/explicit-module-boundary-types": "off",
"@typescript-eslint/no-explicit-any": "off",
"@typescript-eslint/no-unsafe-assignment": "off",
"@typescript-eslint/no-unsafe-call": "off",
"@typescript-eslint/no-unsafe-member-access": "off",
"@typescript-eslint/no-unsafe-return": "off",
"@typescript-eslint/no-unused-vars": "off",
"@typescript-eslint/no-var-requires": "off",
"@typescript-eslint/prefer-regexp-exec": "off",
"@typescript-eslint/require-await": "off",
"@typescript-eslint/restrict-template-expressions": "off",
"eslint-comments/no-use": "off",
"func-style": "off",
"github/array-foreach": "off",
"github/no-then": "off",
"import/no-extraneous-dependencies": "off",
"no-shadow": "off",
"no-sparse-arrays": "off",
"no-throw-literal": "off",
"no-useless-escape": "off",
"sort-imports": "off"
}
}]
}
5 changes: 4 additions & 1 deletion .github/workflows/codeql.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
name: "CodeQL action"

on: [push, pull_request]
on:
push:
branches: [main, v1]
pull_request:

jobs:
build:
Expand Down
19 changes: 11 additions & 8 deletions .github/workflows/integration-testing.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
name: "Integration Testing"

on: [push, pull_request]
on:
push:
branches: [main, v1]
pull_request:

jobs:
multi-language-repo_test-autodetect-languages:
Expand Down Expand Up @@ -150,7 +153,7 @@ jobs:
- uses: ./../action/analyze
env:
TEST_MODE: true

runner-analyze-javascript-ubuntu:
runs-on: ubuntu-latest

Expand All @@ -176,7 +179,7 @@ jobs:
runner/dist/codeql-runner-linux analyze --repository $GITHUB_REPOSITORY --commit $GITHUB_SHA --ref $GITHUB_REF --github-url $GITHUB_SERVER_URL --github-auth ${{ github.token }}
env:
TEST_MODE: true

runner-analyze-javascript-windows:
runs-on: windows-latest

Expand All @@ -198,7 +201,7 @@ jobs:
runner/dist/codeql-runner-win.exe analyze --repository $Env:GITHUB_REPOSITORY --commit $Env:GITHUB_SHA --ref $Env:GITHUB_REF --github-url $Env:GITHUB_SERVER_URL --github-auth ${{ github.token }}
env:
TEST_MODE: true

runner-analyze-javascript-macos:
runs-on: macos-latest

Expand All @@ -220,7 +223,7 @@ jobs:
runner/dist/codeql-runner-macos analyze --repository $GITHUB_REPOSITORY --commit $GITHUB_SHA --ref $GITHUB_REF --github-url $GITHUB_SERVER_URL --github-auth ${{ github.token }}
env:
TEST_MODE: true

runner-analyze-csharp-ubuntu:
runs-on: ubuntu-latest

Expand Down Expand Up @@ -254,7 +257,7 @@ jobs:
../action/runner/dist/codeql-runner-linux analyze --repository $GITHUB_REPOSITORY --commit $GITHUB_SHA --ref $GITHUB_REF --github-url $GITHUB_SERVER_URL --github-auth ${{ github.token }}
env:
TEST_MODE: true

runner-analyze-csharp-windows:
runs-on: windows-latest

Expand Down Expand Up @@ -325,7 +328,7 @@ jobs:
env:
TEST_MODE: true


runner-analyze-csharp-autobuild-ubuntu:
runs-on: ubuntu-latest

Expand Down Expand Up @@ -358,7 +361,7 @@ jobs:
../action/runner/dist/codeql-runner-linux analyze --repository $GITHUB_REPOSITORY --commit $GITHUB_SHA --ref $GITHUB_REF --github-url $GITHUB_SERVER_URL --github-auth ${{ github.token }}
env:
TEST_MODE: true

runner-analyze-csharp-autobuild-windows:
runs-on: windows-latest

Expand Down
18 changes: 10 additions & 8 deletions .github/workflows/pr-checks.yml
Original file line number Diff line number Diff line change
@@ -1,21 +1,24 @@
name: "PR checks"

on: [push, pull_request]
on:
push:
branches: [main, v1]
pull_request:

jobs:
tslint:
lint-js:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v1
- name: tslint
- uses: actions/checkout@v2
- name: Run Lint
run: npm run-script lint

check-js:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v1
- uses: actions/checkout@v2
- name: Check generated JavaScript
run: |
# Sanity check that repo is clean to start with
Expand All @@ -41,7 +44,7 @@ jobs:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v1
- uses: actions/checkout@v2
- name: Check node modules up to date
run: |
# Sanity check that repo is clean to start with
Expand All @@ -50,7 +53,6 @@ jobs:
>&2 echo "Failed: Repo should be clean before testing!"
exit 1
fi

# Reinstall modules and then clean to remove absolute paths
# Use 'npm ci' instead of 'npm install' as this is intended to be reproducible
npm ci
Expand All @@ -71,6 +73,6 @@ jobs:
runs-on: ${{ matrix.os }}

steps:
- uses: actions/checkout@v1
- uses: actions/checkout@v2
- name: npm run-script test
run: npm run-script test
Loading