Skip to content

Commit d2f6baa

Browse files
committed
chore: clean up stale tooling artifacts and review findings
- Remove eslint-config-prettier and its ESLint extends entry (replaced by oxfmt) - Remove ts-node devDependency (no longer needed after Jest→Vitest) - Remove stale .vscode/launch.json (referenced deleted Jest config) - Remove emitDecoratorMetadata from tsconfig (unnecessary for Angular) - Remove duplicate deserialization-manager.service export in public_api - Remove vestigial standalone param from mockDirective test utility - Widen rxjs peer dependency to ^6.5.5 || ^7.0.0 - Enable @angular-eslint/prefer-standalone lint rule - Add non-standalone renderer backward compatibility test - Add util/ to coverage exclusions (types-only files) - Update README with Angular 21 prerequisites and RxJS 6+7 support
1 parent d167147 commit d2f6baa

File tree

11 files changed

+71
-205
lines changed

11 files changed

+71
-205
lines changed

.eslintrc.json

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,7 @@
1111
"extends": [
1212
"plugin:@typescript-eslint/recommended",
1313
"plugin:@angular-eslint/recommended",
14-
"plugin:@angular-eslint/template/process-inline-templates",
15-
"prettier"
14+
"plugin:@angular-eslint/template/process-inline-templates"
1615
],
1716
"plugins": ["eslint-plugin-no-null", "eslint-plugin-unicorn"],
1817
"rules": {
@@ -28,7 +27,7 @@
2827
"@angular-eslint/no-pipe-impure": "error",
2928
"@angular-eslint/no-queries-metadata-property": "error",
3029
"@angular-eslint/prefer-output-readonly": "error",
31-
"@angular-eslint/prefer-standalone": "off",
30+
"@angular-eslint/prefer-standalone": "error",
3231
"@angular-eslint/use-component-selector": "error",
3332
"@angular-eslint/use-component-view-encapsulation": "error",
3433
"@angular-eslint/prefer-inject": "off",

.vscode/launch.json

Lines changed: 0 additions & 15 deletions
This file was deleted.

README.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,18 +4,19 @@
44

55
## Prerequisites
66

7-
Install Node + NPM
7+
- Angular `^21.0.0`
8+
- Node `^20.19.0 || ^22.12.0 || >=23`
9+
- NPM
810

911
## Setup
1012

1113
`npm install`
1214

1315
## RxJS Compatibility
1416

15-
`@hypertrace/hyperdash-angular` currently uses `rxjs` `^6.5.5`.
17+
`@hypertrace/hyperdash-angular` supports both RxJS 6 (`^6.5.5`) and RxJS 7 (`^7.0.0`).
1618

17-
The library code is kept compatible with both RxJS 6 and RxJS 7 import styles where practical, but
18-
published compatibility currently follows `@hypertrace/hyperdash` peer requirements.
19+
The library code is kept compatible with both RxJS 6 and RxJS 7 import styles where practical.
1920

2021
## Development server
2122

angular.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,12 @@
122122
"runnerConfig": "vitest.config.ts",
123123
"coverage": true,
124124
"coverageInclude": ["projects/hyperdash-angular/src/**/*.ts"],
125-
"coverageExclude": ["**/*.module.ts", "**/public_api.ts", "projects/hyperdash-angular/src/test/**"],
125+
"coverageExclude": [
126+
"**/*.module.ts",
127+
"**/public_api.ts",
128+
"projects/hyperdash-angular/src/test/**",
129+
"projects/hyperdash-angular/src/util/**"
130+
],
126131
"reporters": ["default", ["junit", { "outputFile": "test-results/hyperdash-angular/results.xml" }]]
127132
}
128133
},

package-lock.json

Lines changed: 0 additions & 175 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@
5252
"commitizen": "^4.3.1",
5353
"cz-conventional-changelog": "^3.3.0",
5454
"eslint": "^9.37.0",
55-
"eslint-config-prettier": "^10.1.8",
5655
"eslint-plugin-import": "2.32.0",
5756
"eslint-plugin-jsdoc": "61.2.1",
5857
"eslint-plugin-no-null": "1.0.2",
@@ -63,7 +62,6 @@
6362
"ng-packagr": "^21.2.0",
6463
"oxfmt": "^0.35.0",
6564
"semantic-release": "^19.0.5",
66-
"ts-node": "~10.9.2",
6765
"typescript": "~5.9.3",
6866
"vitest": "^4.0.18"
6967
},

projects/hyperdash-angular/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
"@angular/common": "^21.0.0",
1818
"@angular/core": "^21.0.0",
1919
"@angular/platform-browser": "^21.0.0",
20-
"rxjs": "^6.5.5",
20+
"rxjs": "^6.5.5 || ^7.0.0",
2121
"lodash-es": "^4.17.21",
2222
"@hypertrace/hyperdash": "^1.3.0"
2323
},

projects/hyperdash-angular/src/public_api.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ export * from './injectable-wrappers/model-manager.service';
2424
export * from './injectable-wrappers/serialization/serialization-manager.service';
2525
export * from './injectable-wrappers/deserialization/deserialization-manager.service';
2626
export * from './injectable-wrappers/serialization/model-serializer.service';
27-
export * from './injectable-wrappers/deserialization/deserialization-manager.service';
2827
export * from './injectable-wrappers/deserialization/model-deserializer.service';
2928

3029
export { MODEL_API, ModelInject } from './model/decorators/model-inject.service';

0 commit comments

Comments
 (0)