Skip to content

Commit

Permalink
feat(eslint-www): unused vars are error now
Browse files Browse the repository at this point in the history
  • Loading branch information
sixmen committed Dec 21, 2023
1 parent 9933d79 commit 5414f0b
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 5 deletions.
1 change: 1 addition & 0 deletions .github/workflows/eslint-config-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ jobs:
with:
version: npm run changeset version
publish: npm run release
title: 'chore(eslint): release x.y.z'
cwd: ./TypeScript/eslint
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/eslint-config-www-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,11 @@ jobs:
run: npm install

- name: Create Release Pull Request or Publish to npm
id: changesets
uses: changesets/action@v1
with:
version: npm run changeset version
publish: npm run release
title: 'chore(eslint-www): release x.y.z'
cwd: ./TypeScript/eslint-www
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Expand Down
3 changes: 2 additions & 1 deletion TypeScript/eslint-www/.changeset/config.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
{
"$schema": "https://unpkg.com/@changesets/config@1.6.0/schema.json",
"$schema": "https://unpkg.com/@changesets/config@3.0.0/schema.json",
"changelog": "@changesets/cli/changelog",
"commit": false,
"fixed": [],
"linked": [],
"access": "public",
"baseBranch": "main",
Expand Down
5 changes: 5 additions & 0 deletions TypeScript/eslint-www/.changeset/fuzzy-colts-tie.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@croquiscom/eslint-config-www': minor
---

unused vars are error now
4 changes: 2 additions & 2 deletions TypeScript/eslint-www/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -336,10 +336,10 @@ module.exports = {
// 불필요한 조건문 / operators를 사용하지 않는다
'@typescript-eslint/no-unnecessary-condition': ['error', { allowConstantLoopConditions: true }],

// 사용하지 않는 변수를 경고한다. 단 _로 시작하는 변수는 무시한다.
// 사용하지 않는 변수를 허용하지 않는다. 단 _로 시작하는 변수는 무시한다.
'no-unused-vars': 'off',
'@typescript-eslint/no-unused-vars': [
'warn',
'error',
{
ignoreRestSiblings: true,
varsIgnorePattern: '^_',
Expand Down
2 changes: 1 addition & 1 deletion TypeScript/eslint-www/package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "@croquiscom/eslint-config-www",
"version": "2.5.0",
"description": "ESLint configuration used by Croquiscom www",
"description": "ESLint configuration used by Kakaostyle web projects",
"main": "index.js",
"repository": {
"type": "git",
Expand Down

0 comments on commit 5414f0b

Please sign in to comment.