Skip to content

Commit

Permalink
update prettier config
Browse files Browse the repository at this point in the history
  • Loading branch information
scandar committed Nov 11, 2024
1 parent 0b0a0b8 commit b80d7f9
Show file tree
Hide file tree
Showing 10 changed files with 183 additions and 181 deletions.
142 changes: 71 additions & 71 deletions .github/linters/eslint.config.mjs
Original file line number Diff line number Diff line change
@@ -1,111 +1,111 @@
import { FlatCompat } from '@eslint/eslintrc'
import js from '@eslint/js'
import typescriptEslint from '@typescript-eslint/eslint-plugin'
import tsParser from '@typescript-eslint/parser'
import vitest from '@vitest/eslint-plugin'
import globals from 'globals'
import path from 'node:path'
import { fileURLToPath } from 'node:url'
import { FlatCompat } from "@eslint/eslintrc";
import js from "@eslint/js";
import typescriptEslint from "@typescript-eslint/eslint-plugin";
import tsParser from "@typescript-eslint/parser";
import vitest from "@vitest/eslint-plugin";
import globals from "globals";
import path from "node:path";
import { fileURLToPath } from "node:url";

const __filename = fileURLToPath(import.meta.url)
const __dirname = path.dirname(__filename)
const __filename = fileURLToPath(import.meta.url);
const __dirname = path.dirname(__filename);
const compat = new FlatCompat({
baseDirectory: __dirname,
recommendedConfig: js.configs.recommended,
allConfig: js.configs.all
})
allConfig: js.configs.all,
});

export default [
{
ignores: [
'!**/.*',
'node_modules',
'dist',
'coverage',
'lib',
'**/*.json',
'eslint.config.mjs',
'vitest.config.mjs',
'dist/**/*.ts',
'dist/**',
'**/*.mjs',
'**/*.js'
]
"!**/.*",
"node_modules",
"dist",
"coverage",
"lib",
"**/*.json",
"eslint.config.mjs",
"vitest.config.mjs",
"dist/**/*.ts",
"dist/**",
"**/*.mjs",
"**/*.js",
],
},
...compat.extends(
'eslint:recommended',
'plugin:@typescript-eslint/recommended-type-checked'
"eslint:recommended",
"plugin:@typescript-eslint/recommended-type-checked",
),
{
plugins: {
'@typescript-eslint': typescriptEslint
"@typescript-eslint": typescriptEslint,
},

languageOptions: {
globals: {
...globals.node,
Atomics: 'readonly',
SharedArrayBuffer: 'readonly'
Atomics: "readonly",
SharedArrayBuffer: "readonly",
},

parser: tsParser,
ecmaVersion: 2023,
sourceType: 'module',
sourceType: "module",

parserOptions: {
project: ['./.github/linters/tsconfig.json', './tsconfig.json']
}
project: ["./.github/linters/tsconfig.json", "./tsconfig.json"],
},
},

rules: {
camelcase: 'off',
'eslint-comments/no-use': 'off',
'eslint-comments/no-unused-disable': 'off',
'i18n-text/no-en': 'off',
'import/no-namespace': 'off',
'no-console': 'off',
semi: 'off',
'@typescript-eslint/array-type': 'error',
'@typescript-eslint/consistent-type-assertions': 'error',
camelcase: "off",
"eslint-comments/no-use": "off",
"eslint-comments/no-unused-disable": "off",
"i18n-text/no-en": "off",
"import/no-namespace": "off",
"no-console": "off",
semi: "off",
"@typescript-eslint/array-type": "error",
"@typescript-eslint/consistent-type-assertions": "error",

'@typescript-eslint/explicit-member-accessibility': [
'error',
"@typescript-eslint/explicit-member-accessibility": [
"error",
{
accessibility: 'no-public'
}
accessibility: "no-public",
},
],

'@typescript-eslint/explicit-function-return-type': [
'error',
"@typescript-eslint/explicit-function-return-type": [
"error",
{
allowExpressions: true
}
allowExpressions: true,
},
],

'@typescript-eslint/no-empty-interface': 'error',
'@typescript-eslint/no-extraneous-class': 'error',
'@typescript-eslint/no-inferrable-types': 'error',
'@typescript-eslint/no-non-null-assertion': 'warn',
'@typescript-eslint/no-unnecessary-qualifier': 'error',
'@typescript-eslint/no-useless-constructor': 'error',
'@typescript-eslint/no-var-requires': 'error',
'@typescript-eslint/prefer-for-of': 'warn',
'@typescript-eslint/prefer-function-type': 'warn',
'@typescript-eslint/prefer-includes': 'error',
'@typescript-eslint/prefer-string-starts-ends-with': 'error',
'@typescript-eslint/promise-function-async': 'error',
'@typescript-eslint/require-array-sort-compare': 'error',
'@typescript-eslint/space-before-function-paren': 'off'
}
"@typescript-eslint/no-empty-interface": "error",
"@typescript-eslint/no-extraneous-class": "error",
"@typescript-eslint/no-inferrable-types": "error",
"@typescript-eslint/no-non-null-assertion": "warn",
"@typescript-eslint/no-unnecessary-qualifier": "error",
"@typescript-eslint/no-useless-constructor": "error",
"@typescript-eslint/no-var-requires": "error",
"@typescript-eslint/prefer-for-of": "warn",
"@typescript-eslint/prefer-function-type": "warn",
"@typescript-eslint/prefer-includes": "error",
"@typescript-eslint/prefer-string-starts-ends-with": "error",
"@typescript-eslint/promise-function-async": "error",
"@typescript-eslint/require-array-sort-compare": "error",
"@typescript-eslint/space-before-function-paren": "off",
},
},
{
files: ['__tests__/**'],
files: ["__tests__/**"],
plugins: {
vitest
vitest,
},
rules: {
...vitest.configs.recommended.rules, // you can also use vitest.configs.all.rules to enable all rules
'vitest/max-nested-describe': ['error', { max: 3 }] // you can also modify rules' behavior using option like this
}
}
]
"vitest/max-nested-describe": ["error", { max: 3 }], // you can also modify rules' behavior using option like this
},
},
];
7 changes: 3 additions & 4 deletions .prettierrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,10 @@
"printWidth": 80,
"tabWidth": 2,
"useTabs": false,
"semi": false,
"singleQuote": true,
"semi": true,
"singleQuote": false,
"quoteProps": "as-needed",
"jsxSingleQuote": false,
"trailingComma": "none",
"trailingComma": "all",
"bracketSpacing": true,
"bracketSameLine": true,
"arrowParens": "avoid",
Expand Down
26 changes: 13 additions & 13 deletions action.yml
Original file line number Diff line number Diff line change
@@ -1,32 +1,32 @@
name: 'ownyourcode'
description: 'Identify orphan files missing codeowners'
author: 'Girges Scandar'
name: "ownyourcode"
description: "Identify orphan files missing codeowners"
author: "Girges Scandar"

# Add your action's branding here. This will appear on the GitHub Marketplace.
branding:
icon: 'users'
color: 'orange'
icon: "users"
color: "orange"

inputs:
fail-on-missing-codeowners:
description: 'Fail CI if there are files without owners'
default: 'true'
description: "Fail CI if there are files without owners"
default: "true"
required: false
github_token:
description: 'Github Token'
description: "Github Token"
required: true
pr-number:
description: 'Scan only the files modified in the PR'
description: "Scan only the files modified in the PR"
required: false
branch:
description: 'Scan all files in the specified branch'
description: "Scan all files in the specified branch"
required: false
codeowners_path:
description: 'Path to the codeowners file'
default: 'CODEOWNERS'
description: "Path to the codeowners file"
default: "CODEOWNERS"
required: false
codeartifact-auth-token:
description: 'Token for the registry'
description: "Token for the registry"
required: true
# outputs:
# total_orphan_files:
Expand Down
18 changes: 9 additions & 9 deletions src/index.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,16 @@
* Unit tests for the action's entrypoint, src/index.ts
*/

import { describe, expect, it, vi } from 'vitest'
import * as main from './main'
import { describe, expect, it, vi } from "vitest";
import * as main from "./main";

// Mock the action's entrypoint
const runMock = vi.spyOn(main, 'run').mockImplementation(async () => {})
const runMock = vi.spyOn(main, "run").mockImplementation(async () => {});

describe('index', () => {
it('calls run when imported', async () => {
await import('./index.js')
describe("index", () => {
it("calls run when imported", async () => {
await import("./index.js");

expect(runMock).toHaveBeenCalled()
})
})
expect(runMock).toHaveBeenCalled();
});
});
4 changes: 2 additions & 2 deletions src/index.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
/**
* The entrypoint for the action.
*/
import { run } from './main'
import { run } from "./main";

// eslint-disable-next-line @typescript-eslint/no-floating-promises
run()
run();
Loading

0 comments on commit b80d7f9

Please sign in to comment.