chore: add prettier (#91)

Co-authored-by: Daniel Kimmich <[email protected]>

Author: json-derulo

.eslintrc.json

@@ -21,19 +21,13 @@
         "no-unused-vars": "off",
         "@typescript-eslint/no-unused-vars": "error",
         "@typescript-eslint/prefer-literal-enum-member": "off",
-        "comma-dangle": [
-          "error",
-          "always-multiline"
-        ]
+        "comma-dangle": ["error", "always-multiline"]
       }
     },
     {
       "files": "*.html",
-      "extends": [
-        "plugin:@angular-eslint/template/recommended"
-      ],
-      "rules": {
-      }
+      "extends": ["plugin:@angular-eslint/template/recommended"],
+      "rules": {}
     }
   ]
 }

.github/ISSUE_TEMPLATE/bug_report.md

@@ -1,17 +1,17 @@
 ---
 name: Bug report
 about: Create a report to help us improve
-title: ''
+title: ""
 labels: bug
 assignees: json-derulo
-
 ---
 
 **Describe the bug**
 A clear and concise description of what the bug is.
 
 **To Reproduce**
 Steps to reproduce the behavior:
+
 1. Go to '...'
 2. Click on '....'
 3. Scroll down to '....'
@@ -21,9 +21,10 @@ Steps to reproduce the behavior:
 A clear and concise description of what you expected to happen.
 
 **Versions**
- - OS: [e.g. iOS]
- - Browser [e.g. chrome, safari]
- - Version [e.g. 1.0.0]
+
+- OS: [e.g. iOS]
+- Browser [e.g. chrome, safari]
+- Version [e.g. 1.0.0]
 
 **Additional context**
 Add any other context about the problem here.

.github/ISSUE_TEMPLATE/feature_request.md

@@ -1,10 +1,9 @@
 ---
 name: Feature request
 about: Suggest an idea for this project
-title: ''
-labels: ''
-assignees: ''
-
+title: ""
+labels: ""
+assignees: ""
 ---
 
 **Description**

.github/workflows/verify-build.yml

@@ -2,7 +2,7 @@ name: Verify build
 
 on:
   pull_request_target:
-    types: [ opened, synchronize, reopened ]
+    types: [opened, synchronize, reopened]
 
 jobs:
   build:
@@ -46,3 +46,17 @@ jobs:
 
       - name: test
         run: npm run test -- --watch=false --browsers=ChromeHeadless
+
+  prettier:
+    runs-on: ubuntu-20.04
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+
+      - name: install
+        run: npm ci
+
+      - name: prettier
+        run: npx prettier . --check

.vscode/extensions.json

@@ -1,6 +1,4 @@
 {
   // For more information, visit: https://go.microsoft.com/fwlink/?linkid=827846
-  "recommendations": [
-    "angular.ng-template"
-  ]
+  "recommendations": ["angular.ng-template"]
 }

README.md

@@ -28,10 +28,7 @@
           "builder": "@angular-devkit/build-angular:karma",
           "options": {
             "tsConfig": "projects/angular-ecmascript-intl/tsconfig.spec.json",
-            "polyfills": [
-              "zone.js",
-              "zone.js/testing"
-            ]
+            "polyfills": ["zone.js", "zone.js/testing"]
           }
         }
       }
@@ -53,9 +50,7 @@
             "outputPath": "dist/angular-intl-demo",
             "index": "projects/angular-intl-demo/src/index.html",
             "main": "projects/angular-intl-demo/src/main.ts",
-            "polyfills": [
-              "zone.js"
-            ],
+            "polyfills": ["zone.js"],
             "tsConfig": "projects/angular-intl-demo/tsconfig.app.json",
             "inlineStyleLanguage": "scss",
             "assets": [

angular.json

@@ -10,7 +10,8 @@
     "watch": "ng build --watch --configuration development",
     "test": "ng test --code-coverage=true",
     "lint": "eslint --ignore-path=.gitignore .",
-    "release": "release-it"
+    "release": "release-it",
+    "prettier": "prettier . --write"
   },
   "private": true,
   "dependencies": {
@@ -54,6 +55,7 @@
     "karma-jasmine": "~5.1.0",
     "karma-jasmine-html-reporter": "~2.1.0",
     "ng-packagr": "~16.1.0",
+    "prettier": "~3.0.0",
     "typescript": "~5.1.0"
   }
 }

package-lock.json

@@ -1,4 +1,6 @@
-import {InjectionToken} from "@angular/core";
-import {IntlCountryPipeOptions} from "./intl-country.pipe";
+import { InjectionToken } from '@angular/core';
+import { IntlCountryPipeOptions } from './intl-country.pipe';
 
-export const INTL_COUNTRY_PIPE_DEFAULT_OPTIONS = new InjectionToken<Omit<IntlCountryPipeOptions, 'locale'>>('IntlCountryPipeDefaultOptions');
+export const INTL_COUNTRY_PIPE_DEFAULT_OPTIONS = new InjectionToken<
+  Omit<IntlCountryPipeOptions, 'locale'>
+>('IntlCountryPipeDefaultOptions');

package.json

@@ -1,7 +1,7 @@
-import {IntlCountryPipe} from './intl-country.pipe';
-import {TestBed} from "@angular/core/testing";
-import {INTL_LOCALES} from "../locale";
-import {INTL_COUNTRY_PIPE_DEFAULT_OPTIONS} from "./intl-country-pipe-default-options";
+import { IntlCountryPipe } from './intl-country.pipe';
+import { TestBed } from '@angular/core/testing';
+import { INTL_LOCALES } from '../locale';
+import { INTL_COUNTRY_PIPE_DEFAULT_OPTIONS } from './intl-country-pipe-default-options';
 
 describe('IntlCountryPipe', () => {
   let testUnit: IntlCountryPipe;
@@ -58,7 +58,7 @@ describe('IntlCountryPipe', () => {
     });
 
     it('should fall back to the browser default locale', () => {
-      TestBed.configureTestingModule({providers: [IntlCountryPipe]});
+      TestBed.configureTestingModule({ providers: [IntlCountryPipe] });
 
       const result1 = TestBed.inject(IntlCountryPipe).transform('US');
       const result2 = new IntlCountryPipe(navigator.language).transform('US');
@@ -86,7 +86,9 @@ describe('IntlCountryPipe', () => {
       });
       testUnit = TestBed.inject(IntlCountryPipe);
 
-      expect(testUnit.transform('DE', {type: 'language'})).toEqual('Deutschland');
+      expect(testUnit.transform('DE', { type: 'language' })).toEqual(
+        'Deutschland',
+      );
     });
   });
 
@@ -102,6 +104,8 @@ describe('IntlCountryPipe', () => {
     });
     testUnit = TestBed.inject(IntlCountryPipe);
 
-    expect(testUnit.transform('US', {locale: 'de-DE'})).toEqual('Vereinigte Staaten');
+    expect(testUnit.transform('US', { locale: 'de-DE' })).toEqual(
+      'Vereinigte Staaten',
+    );
   });
 });

projects/angular-ecmascript-intl/src/lib/country/intl-country-pipe-default-options.ts

@@ -1,36 +1,49 @@
-import {Inject, Optional, Pipe, PipeTransform} from '@angular/core';
-import {INTL_LOCALES} from "../locale";
-import {INTL_COUNTRY_PIPE_DEFAULT_OPTIONS} from "./intl-country-pipe-default-options";
-import {IntlPipeOptions} from "../intl-pipe-options";
+import { Inject, Optional, Pipe, PipeTransform } from '@angular/core';
+import { INTL_LOCALES } from '../locale';
+import { INTL_COUNTRY_PIPE_DEFAULT_OPTIONS } from './intl-country-pipe-default-options';
+import { IntlPipeOptions } from '../intl-pipe-options';
 
-export type IntlCountryPipeOptions = Omit<Partial<Intl.DisplayNamesOptions>, 'languageDisplay'> & IntlPipeOptions;
+export type IntlCountryPipeOptions = Omit<
+  Partial<Intl.DisplayNamesOptions>,
+  'languageDisplay'
+> &
+  IntlPipeOptions;
 
 @Pipe({
   name: 'intlCountry',
   standalone: true,
 })
 export class IntlCountryPipe implements PipeTransform {
+  constructor(
+    @Optional()
+    @Inject(INTL_LOCALES)
+    readonly locale?: string | string[] | null,
+    @Optional()
+    @Inject(INTL_COUNTRY_PIPE_DEFAULT_OPTIONS)
+    readonly defaultOptions?: Omit<IntlCountryPipeOptions, 'locale'> | null,
+  ) {}
 
-  constructor(@Optional() @Inject(INTL_LOCALES) readonly locale?: string | string[] | null,
-              @Optional() @Inject(INTL_COUNTRY_PIPE_DEFAULT_OPTIONS) readonly defaultOptions?: Omit<IntlCountryPipeOptions, 'locale'> | null) {
-  }
-
-  transform(value: string | null | undefined, options?: IntlCountryPipeOptions): string | null {
+  transform(
+    value: string | null | undefined,
+    options?: IntlCountryPipeOptions,
+  ): string | null {
     if (!value) {
       return null;
     }
 
-    const {locale, ...intlOptions} = options ?? {};
+    const { locale, ...intlOptions } = options ?? {};
 
     try {
-      return new Intl.DisplayNames(locale ?? this.locale ?? undefined, {
-        ...this.defaultOptions, ...intlOptions,
-        type: 'region',
-      }).of(value) ?? null;
+      return (
+        new Intl.DisplayNames(locale ?? this.locale ?? undefined, {
+          ...this.defaultOptions,
+          ...intlOptions,
+          type: 'region',
+        }).of(value) ?? null
+      );
     } catch (e) {
       console.error('Error while transforming the country', e);
       return null;
     }
   }
-
 }

projects/angular-ecmascript-intl/src/lib/country/intl-country.pipe.spec.ts

@@ -1,4 +1,6 @@
-import {InjectionToken} from "@angular/core";
-import {IntlCurrencyPipeOptions} from "./intl-currency.pipe";
+import { InjectionToken } from '@angular/core';
+import { IntlCurrencyPipeOptions } from './intl-currency.pipe';
 
-export const INTL_CURRENCY_PIPE_DEFAULT_OPTIONS = new InjectionToken<Omit<IntlCurrencyPipeOptions, 'locale'>>('IntlCurrencyPipeDefaultOptions');
+export const INTL_CURRENCY_PIPE_DEFAULT_OPTIONS = new InjectionToken<
+  Omit<IntlCurrencyPipeOptions, 'locale'>
+>('IntlCurrencyPipeDefaultOptions');