Skip to content

Conversation

@mariusmarin-dev
Copy link

No description provided.

@sonarqubecloud
Copy link

@github-actions
Copy link

Lines Statements Branches Functions
Coverage: 51%
51.4% (9261/18017) 35% (4563/13036) 40.88% (1559/3813)

Copy link

@Parth097 Parth097 left a comment

Choose a reason for hiding this comment

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

Just need to look at the disabling of eslint issue, apart from that looks good to me :)

followedAtLeastOneSoftLink?: boolean;
}

// eslint-disable-next-line no-var

Choose a reason for hiding this comment

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

Is there is good enough reason to disable es-lint here?
Is there another way which es-lint will accept this?

Copy link
Author

Choose a reason for hiding this comment

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

well, it doesn't like it and is not an issue

Choose a reason for hiding this comment

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

So we enforce es-lint so that we meet some standards within the codebase.
I know it is really annoying but we need to make sure we dont disable eslint otherwise there is no point having it in the first place.

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