Add format script, format repo + tests, check format in ci (#128)

* Add format + check to ci

* run formatting on testing and remove from ignore

.github/workflows/test-on-pr.yml

@@ -1,4 +1,4 @@
-name: Unit Test Pull Request
+name: Check Pull Request
 on:
   pull_request:
     branches: [master]
@@ -16,3 +16,12 @@ jobs:
           node-version: ${{ matrix.node-version }}
       - run: yarn
       - run: yarn test
+  check-format:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: '16.x'
+      - run: yarn
+      - run: yarn format:check

.prettierrc

@@ -1,7 +1,5 @@
 {
-  "_docs": "https://prettier.io/docs/en/options.html",
   "tabWidth": 2,
-  "tabs": false,
   "semi": false,
   "singleQuote": true,
   "printWidth": 80,

package.json

@@ -24,7 +24,9 @@
     "test": "jest --coverage",
     "check": "yarn lint && yarn test && yarn build",
     "build": "yarn tsc --noEmit && yarn rollup -c",
-    "lint": "tslint -p tsconfig.json"
+    "lint": "tslint -p tsconfig.json",
+    "format": "prettier --write \"./**/*.ts\"",
+    "format:check": "prettier --check \"**/*.ts\" --ignore-unknown"
   },
   "devDependencies": {
     "@rollup/plugin-typescript": "^8.5.0",

src/array.ts

@@ -412,12 +412,12 @@ export const diff = <T>(
  * If n > 0 items will shift n steps to the right
  * If n < 0 items will shift n steps to the left
  */
- export function shift<T>(arr: Array<T>, n: number) {
-  if (arr.length === 0) return arr;
+export function shift<T>(arr: Array<T>, n: number) {
+  if (arr.length === 0) return arr
 
-  const shiftNumber = n % arr.length;
+  const shiftNumber = n % arr.length
 
-  if(shiftNumber === 0) return arr;
+  if (shiftNumber === 0) return arr
 
-  return [...arr.slice(-shiftNumber, arr.length), ...arr.slice(0, -shiftNumber)];
+  return [...arr.slice(-shiftNumber, arr.length), ...arr.slice(0, -shiftNumber)]
 }

src/index.ts

@@ -20,11 +20,11 @@ export {
   replace,
   replaceOrAppend,
   select,
+  shift,
   sift,
   sort,
   sum,
-  unique,
-  shift
+  unique
 } from './array'
 export {
   defer,

src/object.ts

@@ -156,9 +156,8 @@ export const pick = <T extends Record<string, unknown>, TKeys extends keyof T>(
 ): Pick<T, TKeys> => {
   if (!obj) return {} as Pick<T, TKeys>
   return keys.reduce((acc, key) => {
-    if(obj.hasOwnProperty(key))
-      acc[key] = obj[key]
-    return acc;
+    if (obj.hasOwnProperty(key)) acc[key] = obj[key]
+    return acc
   }, {} as Pick<T, TKeys>)
 }
 

src/string.ts

@@ -18,10 +18,11 @@ export const capitalize = (str: string): string => {
  * camel('helloWorld') -> 'helloWorld'
  */
 export const camel = (str: string): string => {
-  const parts = str
-    ?.replace(/([A-Z])+/g, capitalize)
-    ?.split(/(?=[A-Z])|[\.\-\s_]/)
-    .map(x => x.toLowerCase()) ?? []
+  const parts =
+    str
+      ?.replace(/([A-Z])+/g, capitalize)
+      ?.split(/(?=[A-Z])|[\.\-\s_]/)
+      .map(x => x.toLowerCase()) ?? []
   if (parts.length === 0) return ''
   if (parts.length === 1) return parts[0]
   return parts.reduce((acc, part) => {
@@ -37,10 +38,11 @@ export const camel = (str: string): string => {
  * snake('helloWord') -> 'hello_world'
  */
 export const snake = (str: string): string => {
-  const parts = str
-    ?.replace(/([A-Z])+/g, capitalize)
-    .split(/(?=[A-Z])|[\.\-\s_]/)
-    .map(x => x.toLowerCase()) ?? []
+  const parts =
+    str
+      ?.replace(/([A-Z])+/g, capitalize)
+      .split(/(?=[A-Z])|[\.\-\s_]/)
+      .map(x => x.toLowerCase()) ?? []
   if (parts.length === 0) return ''
   if (parts.length === 1) return parts[0]
   return parts.reduce((acc, part) => {
@@ -56,10 +58,11 @@ export const snake = (str: string): string => {
  * dash('helloWord') -> 'hello-word'
  */
 export const dash = (str: string): string => {
-  const parts = str
-    ?.replace(/([A-Z])+/g, capitalize)
-    ?.split(/(?=[A-Z])|[\.\-\s_]/)
-    .map(x => x.toLowerCase()) ?? []
+  const parts =
+    str
+      ?.replace(/([A-Z])+/g, capitalize)
+      ?.split(/(?=[A-Z])|[\.\-\s_]/)
+      .map(x => x.toLowerCase()) ?? []
   if (parts.length === 0) return ''
   if (parts.length === 1) return parts[0]
   return parts.reduce((acc, part) => {