Skip to content
#

lint

Here are 878 public repositories matching this topic...

stylelint
tomyo
tomyo commented Mar 19, 2021

Clearly describe the bug
The rule length-zero-no-unit is applying fixes inside level 4 math functions.

This test exercises exactly what it shouldn't happen:

https://github.com/tomyo/stylelint/blob/eb419ef754ac442f9f3b9f7164fccb938b99adb3/lib/rules/length-zero-no-unit/__tests__/index.js#L332

Why?
It converts this:

padding: calc(var(--foo, 0px) + 10px);

Into this:

azu
azu commented Nov 1, 2019

Current

export declare type TextlintRuleReportHandler = {
    [P in ASTNodeTypes]?: (node: TypeofTxtNode<P>) => void | Promise<any>;
} & {
    [index: string]: (node: any) => void | Promise<any>;
};
/**
 * Textlint rule reporter function
 */
export declare type TextlintRuleReporter<T extends object = {}> = (context: Readonly<TextlintRuleContext>, options?: TextlintRuleOptions<T>) 
cfn_nag
bquorning
bquorning commented Mar 23, 2021

I'm seeing double/instance_double/class_double/object_double/spy etc. being called with sometimes a string, sometimes a symbol (perhaps because the rspec-mocks documentation explicitly allows both), and of course sometimes a constant referring to a class or module.

I think mixing string and symbol arguments is a bit of a mess, so I would propose making a new cop that enforces either one or the

Improve this page

Add a description, image, and links to the lint topic page so that developers can more easily learn about it.

Curate this topic

Add this topic to your repo

To associate your repository with the lint topic, visit your repo's landing page and select "manage topics."

Learn more