Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions .github/matchers/actionlint.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
{
"problemMatcher": [
{
"owner": "actionlint",
"pattern": [
{
"regexp": "^(?:\\x1b\\[\\d+m)?(.+?)(?:\\x1b\\[\\d+m)*:(?:\\x1b\\[\\d+m)*(\\d+)(?:\\x1b\\[\\d+m)*:(?:\\x1b\\[\\d+m)*(\\d+)(?:\\x1b\\[\\d+m)*: (?:\\x1b\\[\\d+m)*(.+?)(?:\\x1b\\[\\d+m)* \\[(.+?)\\]$",
"file": 1,
"line": 2,
"column": 3,
"message": 4,
"code": 5
}
]
}
]
}
32 changes: 25 additions & 7 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,7 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
include:
- os: ubuntu-latest
target-folder: drop-linux
- os: windows-latest
target-folder: drop
os: [ubuntu-latest, windows-latest]
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

target-folder was dead code accidentally left behind


steps:
- uses: actions/checkout@v6
Expand All @@ -30,11 +26,33 @@ jobs:
- name: npm ci
run: npm ci

- name: test
run: npm test

- name: lint
run: npm run lint

- name: prettier
run: npm run prettier

- name: test
run: npm test
- if: runner.os == 'Linux'
name: Pack
run: npm pack

- if: runner.os == 'Linux'
name: Upload tarball
uses: actions/upload-artifact@v7
with:
path: "azure-oad-*.tgz"
archive: false

# Content copied from https://raw.githubusercontent.com/rhysd/actionlint/2ab3a12c7848f6c15faca9a92612ef4261d0e370/.github/actionlint-matcher.json
- if: runner.os == 'Linux'
name: Add ActionLint Problem Matcher
run: echo "::add-matcher::.github/matchers/actionlint.json"

- if: runner.os == 'Linux'
name: Lint workflows
uses: docker://rhysd/actionlint:1.7.11
with:
args: -color -verbose
1 change: 1 addition & 0 deletions .prettierignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@
*.jsonc
.prettierrc
azure-pipelines.yml
coverage/
Copy link
Copy Markdown
Member Author

@mikeharder mikeharder Apr 1, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needed now because we test before format

2 changes: 1 addition & 1 deletion eslint.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ module.exports = tseslint.config(
},
{
// Based on https://eslint.org/docs/latest/use/configure/configuration-files#globally-ignoring-files-with-ignores
ignores: ["**/dist", "eslint.config.js"]
ignores: ["**/dist", "eslint.config.js", "coverage/**"]
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needed now because we test before lint

},
{
rules: {
Expand Down
Loading