Skip to content
This repository was archived by the owner on Jul 19, 2025. It is now read-only.

Add support to scala #324

Merged
merged 2 commits into from
Jun 22, 2018
Merged

Add support to scala #324

merged 2 commits into from
Jun 22, 2018

Conversation

filipesperandio
Copy link
Contributor

@filipesperandio filipesperandio commented Jun 21, 2018

@codeclimate-hermes codeclimate-hermes requested review from ale7714 and removed request for codeclimate-hermes June 21, 2018 21:46
@codeclimate-hermes codeclimate-hermes requested review from larkinscott and removed request for codeclimate-hermes June 21, 2018 21:46
@ale7714 ale7714 removed their request for review June 22, 2018 03:21
@filipesperandio filipesperandio requested review from codeclimate-hermes and removed request for larkinscott June 22, 2018 15:27
@codeclimate-hermes codeclimate-hermes requested review from wilson and removed request for codeclimate-hermes June 22, 2018 15:27
LANGUAGE = "scala".freeze
PATTERNS = ["**/*.sc", "**/*.scala"].freeze
DEFAULT_MASS_THRESHOLD = 60
DEFAULT_FILTERS = [].freeze
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you think we need to ignore imports by default? That's a common source of false positives in many languages.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think I haven't hit issues with the current DEFAULT_MASS_THRESHOLD. Improving the spec so I can adjust this! 👍

@filipesperandio filipesperandio merged commit 0646b13 into master Jun 22, 2018
@filipesperandio filipesperandio deleted the scala branch June 22, 2018 19:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants