Skip to content
This repository has been archived by the owner on Oct 10, 2022. It is now read-only.

Enable fp/no-class ESLint rule #251

Merged
merged 1 commit into from
Dec 20, 2020
Merged

Enable fp/no-class ESLint rule #251

merged 1 commit into from
Dec 20, 2020

Conversation

ehmicky
Copy link
Contributor

@ehmicky ehmicky commented Dec 18, 2020

This enables the fp/no-class ESLint rule.

@ehmicky ehmicky added the type: chore work needed to keep the product and development running smoothly label Dec 18, 2020
@ehmicky ehmicky self-assigned this Dec 18, 2020
@@ -5,6 +5,7 @@ const deploy = require('./deploy')
const { getMethods } = require('./methods')
const { getOperations } = require('./operations')

// eslint-disable-next-line fp/no-class
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removing the class would be a major breaking change with little benefit.

@erezrokah erezrokah merged commit f0ac699 into master Dec 20, 2020
@erezrokah erezrokah deleted the lint/no-class branch December 20, 2020 08:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: chore work needed to keep the product and development running smoothly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants