-
-
Notifications
You must be signed in to change notification settings - Fork 757
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
Added packagePattern option to config #329
Conversation
This shows that we need to find a way (Wiki, |
|
||
class NamingConventionSpec : Spek({ | ||
|
||
given("a kt file with crazy naming schemes") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we migrate these test cases over to the other test class to keep the same coverage?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm working on it. I don't know what problem the appveyor build had with my first commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also noticed the appveyor problem in some of my commits. Kotlin.Unit cannot be loaded ^^
It looks like its rdy to merge? |
@arturbosch yes |
I also spotted that we have two tests for the
NamingConventionViolation
rule. I think one should be deleted because they are basically testing the same thing.EDIT: fixes #327