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: part 6 #3674

Merged
merged 1 commit into from
Mar 8, 2022

Conversation

coderaiser
Copy link
Contributor

@coderaiser coderaiser commented Mar 8, 2022

There is surely nothing other than the single purpose of the present moment. A man's whole life is a succession of moment after moment. There will be nothing else to do, and nothing else to pursue. Live being true to the single purpose of the moment.

 (c) Yamamoto Tsunetomo "Hagakure"

image

There is no holidays in the the peaceful swamps since the invasion of hippo herd 🦛.

Anyways 🐊Putout does what it always did: searches ways to simplify code. Outside of the window sounds roar of shells and charms of sirens winding the winds. But 🐊he is OK and here is report of Xterm linting by 🐊v25:

☝️ As usual, any rule can be disabled.

Command used:

putout . --fix

Applied rules:

Current config for 🐊Putout v25:

{
    "rules": {
        "apply-is-array": ["on", {
            "inline": true
        }],
        "remove-useless-new": "off",
        "remove-useless-return": "off",
        "typescript/remove-useless-types-from-constants": "off",
        "remove-useless-type-conversion/with-double-negations": "off",
        "convert-typeof-to-is-type": "off",
        "apply-shorthand-properties": "off",
        "apply-destructuring": "off",
        "apply-numeric-separators": "off",
        "convert-assignment-to-comparison": "off",
        "convert-apply-to-spread": "off",
        "convert-math-pow": "off",
        "convert-for-to-for-of": "off",
        "convert-template-to-string": "off",
        "strict-mode": "off",
        "remove-useless-spread/object": "off",
        "remove-useless-array-constructor": "off",
        "remove-boolean-from-assertions": "off",
        "remove-iife": "off",
        "remove-console": "off",
        "remove-unused-variables": "off",
        "remove-useless-variables": "off",
        "merge-if-statements": "off",
        "promises/add-missing-await": "off",
        "promises/remove-useless-async": "off",
        "simplify-ternary": "off",
        "nodejs/convert-dirname-to-url": "off",
        "try-catch": "off",
        "apply-array-at": "off"
    },
    "match": {
        "*.benchmark.ts": {
            "remove-unused-expressions": "off"
        },
        "SerializeAddon.test.ts": {
            "regexp/optimize": "off"
        }
    },
    "plugins": [
        "apply-shorthand-properties"
    ],
    "ignore": [
        "addons/*/lib/*.js",
        "*.benchmark.js",
        "*.md",
        "*.json",
        "*ignore",
        "*.yml",
        ".npmrc",
        "*.css",
        "out",
        "out-test",
        "demo"
    ]
}

Previous reports:

🐊Have a clear sky and blooming fields!

src/common/parser/EscapeSequenceParser.test.ts Outdated Show resolved Hide resolved
test/api/MouseTracking.api.ts Outdated Show resolved Hide resolved
@Tyriar Tyriar closed this Mar 8, 2022
@Tyriar Tyriar reopened this Mar 8, 2022
coderaiser added a commit to coderaiser/putout that referenced this pull request Mar 8, 2022
Copy link
Member

@Tyriar Tyriar left a comment

Choose a reason for hiding this comment

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

Thanks 👍

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.

3 participants