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

Lint using 🐊Putout #111

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

coderaiser
Copy link
Contributor

@coderaiser coderaiser commented Apr 9, 2022

Hi again! I think CodeshiftComunity does a great job of popularizing codemods, and I want to suggest you work results of a tool which simplifies writing, testing and using codemods.

🐊 Putout Report

image

Among the maxims on Lord Naoshige’s wall there was this one: ”Matters of’ great concern should be treated lightly.” Master lttei commented, “Matters of small concern should be treated seriously.”

(c) Yamamoto Tsunetomo "Hagakure"

Command used:

putout packages --fix

☝️ Any rule can be disabled.

Applied rules:

Current config for putout v25:

{
    "rules": {
        "remove-console": "off",
        "package-json/add-type": "off",
        "strict-mode": "off",
        "nodejs/convert-dirname-to-url": "off"
    },
    "ignore": [
        "tsconfig.json",
        "package.json",
        ".npmignore",
        ".gitignore"
    ]
}

Hope in integration in future :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant