Skip to content

Commit

Permalink
fix: fix Eslint configuration to parse all files.
Browse files Browse the repository at this point in the history
docs: update public documentation to capture linting rules on eslint.
  • Loading branch information
andrueastman committed Nov 4, 2024
1 parent 1e2ea4e commit 9a58d21
Show file tree
Hide file tree
Showing 112 changed files with 1,217 additions and 676 deletions.
49 changes: 48 additions & 1 deletion .github/workflows/build_test_validate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,53 @@ jobs:
USER_ID: ${{secrets.user_id}}
- run: npm run test

code-coverage:
runs-on: ubuntu-latest
strategy:
matrix:
coverage:
- name: "Abstractions"
path: "./packages/abstractions/coverage"
- name: "Authentication Azure"
path: "./packages/authentication/azure/coverage"
- name: "Authentication Spfx"
path: "./packages/authentication/spfx/coverage"
- name: "Http Fetch"
path: "./packages/http/fetch/coverage"
- name: "Serialization Form"
path: "./packages/serialization/form/coverage"
- name: "Serialization JSON"
path: "./packages/serialization/json/coverage"
- name: "Serialization Text"
path: "./packages/serialization/text/coverage"
- name: "Serialization Multipart"
path: "./packages/serialization/multipart/coverage"
- name: "Bundle"
path: "./packages/bundle/coverage"
steps:
- uses: actions/checkout@v4
- name: Use Node.js 22.x
uses: actions/setup-node@v4
with:
node-version: 22.x
- run: npm ci
- run: npm run build
- run: npm run test:coverage
- name: Code Coverage Report ${{ matrix.coverage.name }}
uses: irongut/[email protected]
with:
filename: ${{ matrix.coverage.path }}/cobertura-coverage.xml
format: markdown
hide_branch_rate: false
output: both
badge: true
- name: Archive coverage files
uses: actions/upload-artifact@v4
with:
name: Coverage files for ${{ matrix.coverage.name }}
path: ${{ matrix.coverage.path }}


publish-npm:
if: "${{ github.event_name == 'push' && github.ref == 'refs/heads/main' && contains(github.event.head_commit.message, 'chore: release main') }}"
needs: [build, check-typescript-version-matrix]
Expand Down Expand Up @@ -75,7 +122,7 @@ jobs:
# Use this as a PR status check for GitHub Policy Service instead of individual matrix entry checks.
check-typescript-version-matrix:
runs-on: ubuntu-latest
needs: [build]
needs: [build,code-coverage]
if: always()
steps:
- name: All build matrix options are successful
Expand Down
6 changes: 3 additions & 3 deletions eslint.config.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,9 @@ export default [{
"plugin:@typescript-eslint/stylistic-type-checked",
"prettier",
"@microsoft/eslint-config-msgraph/core",
"plugin:jsdoc/recommended-typescript-error"
), {
plugins: {
jsdoc,
"prefer-arrow": preferArrow,
"@typescript-eslint": typescriptEslint,
header,
Expand All @@ -57,12 +57,12 @@ export default [{
"packages/abstractions/tsconfig.json",
"packages/bundle/tsconfig.json",
"packages/authentication/azure/tsconfig.json",
"packages/authentication/azure/tsconfig.json",
"packages/authentication/spfx/tsconfig.json",
"packages/http/fetch/tsconfig.json",
"packages/serialization/form/tsconfig.json",
"packages/serialization/json/tsconfig.json",
"packages/serialization/text/tsconfig.json",
"packages/serialization/multipart/tsconfig.json",
"packages/authentication/azure/tsconfig.json",
"packages/test/tsconfig.json",
],
},
Expand Down
Loading

0 comments on commit 9a58d21

Please sign in to comment.