Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(eslint-config): refactor repo and update deps #44

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

arsenykruglikov
Copy link
Collaborator

@arsenykruglikov arsenykruglikov commented Sep 16, 2024

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install [email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
# or 
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add [email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]

@arsenykruglikov arsenykruglikov force-pushed the update-eslint branch 7 times, most recently from 56a0949 to 6a17879 Compare September 18, 2024 14:07
"main": "index.js",
"dependencies": {
"@salutejs/eslint-config-base": "*",
"eslint-plugin-salute-rules": "*"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Было так.
"eslint-plugin-salute-rules": "file:./rules"

Теперь так
""eslint-plugin-salute-rules": "*"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

При сборке конфиг теперь не может найти, eslint-plugin-salute-rules .
Наверно правила нужно будет отдельно выложить, чтобы не возникло таких проблем в будущем.

@arsenykruglikov arsenykruglikov force-pushed the update-eslint branch 2 times, most recently from 7b79353 to cf7748a Compare September 23, 2024 09:57
@SeanSilke
Copy link
Contributor

Давай правила тоже загоним по @salutejs/eslint-rules ?

@SeanSilke SeanSilke closed this Sep 24, 2024
@SeanSilke SeanSilke reopened this Sep 24, 2024
@SeanSilke SeanSilke self-assigned this Sep 24, 2024
packages/eslint-plugin-salute-rules/index.js Dismissed Show dismissed Hide dismissed
@arsenykruglikov arsenykruglikov force-pushed the update-eslint branch 5 times, most recently from ded1ca3 to 538feea Compare September 26, 2024 12:26
@arsenykruglikov arsenykruglikov added the major Auto js major release label Sep 26, 2024
@arsenykruglikov arsenykruglikov force-pushed the update-eslint branch 3 times, most recently from 2474491 to 7649487 Compare September 26, 2024 13:42
@arsenykruglikov arsenykruglikov changed the title fix(eslint-config): refactor repo feat(eslint-config): refactor repo and update deps Sep 26, 2024
@arsenykruglikov arsenykruglikov merged commit 0945cf4 into master Sep 26, 2024
5 checks passed
@arsenykruglikov arsenykruglikov deleted the update-eslint branch September 26, 2024 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major Auto js major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants