Skip to content

Commit 9a58d21

Browse files
author
Andrew Omondi
committed
fix: fix Eslint configuration to parse all files.
docs: update public documentation to capture linting rules on eslint.
1 parent 1e2ea4e commit 9a58d21

File tree

112 files changed

+1217
-676
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

112 files changed

+1217
-676
lines changed

.github/workflows/build_test_validate.yml

+48-1
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,53 @@ jobs:
4747
USER_ID: ${{secrets.user_id}}
4848
- run: npm run test
4949

50+
code-coverage:
51+
runs-on: ubuntu-latest
52+
strategy:
53+
matrix:
54+
coverage:
55+
- name: "Abstractions"
56+
path: "./packages/abstractions/coverage"
57+
- name: "Authentication Azure"
58+
path: "./packages/authentication/azure/coverage"
59+
- name: "Authentication Spfx"
60+
path: "./packages/authentication/spfx/coverage"
61+
- name: "Http Fetch"
62+
path: "./packages/http/fetch/coverage"
63+
- name: "Serialization Form"
64+
path: "./packages/serialization/form/coverage"
65+
- name: "Serialization JSON"
66+
path: "./packages/serialization/json/coverage"
67+
- name: "Serialization Text"
68+
path: "./packages/serialization/text/coverage"
69+
- name: "Serialization Multipart"
70+
path: "./packages/serialization/multipart/coverage"
71+
- name: "Bundle"
72+
path: "./packages/bundle/coverage"
73+
steps:
74+
- uses: actions/checkout@v4
75+
- name: Use Node.js 22.x
76+
uses: actions/setup-node@v4
77+
with:
78+
node-version: 22.x
79+
- run: npm ci
80+
- run: npm run build
81+
- run: npm run test:coverage
82+
- name: Code Coverage Report ${{ matrix.coverage.name }}
83+
uses: irongut/CodeCoverageSummary@v1.3.0
84+
with:
85+
filename: ${{ matrix.coverage.path }}/cobertura-coverage.xml
86+
format: markdown
87+
hide_branch_rate: false
88+
output: both
89+
badge: true
90+
- name: Archive coverage files
91+
uses: actions/upload-artifact@v4
92+
with:
93+
name: Coverage files for ${{ matrix.coverage.name }}
94+
path: ${{ matrix.coverage.path }}
95+
96+
5097
publish-npm:
5198
if: "${{ github.event_name == 'push' && github.ref == 'refs/heads/main' && contains(github.event.head_commit.message, 'chore: release main') }}"
5299
needs: [build, check-typescript-version-matrix]
@@ -75,7 +122,7 @@ jobs:
75122
# Use this as a PR status check for GitHub Policy Service instead of individual matrix entry checks.
76123
check-typescript-version-matrix:
77124
runs-on: ubuntu-latest
78-
needs: [build]
125+
needs: [build,code-coverage]
79126
if: always()
80127
steps:
81128
- name: All build matrix options are successful

eslint.config.mjs

+3-3
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,9 @@ export default [{
3434
"plugin:@typescript-eslint/stylistic-type-checked",
3535
"prettier",
3636
"@microsoft/eslint-config-msgraph/core",
37+
"plugin:jsdoc/recommended-typescript-error"
3738
), {
3839
plugins: {
39-
jsdoc,
4040
"prefer-arrow": preferArrow,
4141
"@typescript-eslint": typescriptEslint,
4242
header,
@@ -57,12 +57,12 @@ export default [{
5757
"packages/abstractions/tsconfig.json",
5858
"packages/bundle/tsconfig.json",
5959
"packages/authentication/azure/tsconfig.json",
60-
"packages/authentication/azure/tsconfig.json",
60+
"packages/authentication/spfx/tsconfig.json",
6161
"packages/http/fetch/tsconfig.json",
6262
"packages/serialization/form/tsconfig.json",
6363
"packages/serialization/json/tsconfig.json",
64+
"packages/serialization/text/tsconfig.json",
6465
"packages/serialization/multipart/tsconfig.json",
65-
"packages/authentication/azure/tsconfig.json",
6666
"packages/test/tsconfig.json",
6767
],
6868
},

0 commit comments

Comments
 (0)