Skip to content
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

Add SubjectStub cop #161

Merged
merged 2 commits into from
Aug 7, 2016
Merged

Add SubjectStub cop #161

merged 2 commits into from
Aug 7, 2016

Conversation

backus
Copy link
Collaborator

@backus backus commented Aug 6, 2016

No description provided.

@backus backus mentioned this pull request Aug 6, 2016
@backus backus force-pushed the feature/Stubject branch 5 times, most recently from b2cfa7e to d050adb Compare August 6, 2016 08:23
@backus
Copy link
Collaborator Author

backus commented Aug 6, 2016

@andyw8 not ready for prime time at all but this should give you an idea of the bare minimum complexity for a cop like this

@backus backus force-pushed the feature/Stubject branch 3 times, most recently from c8dc02c to 8208d81 Compare August 7, 2016 03:46
@backus backus changed the title [Experiment] Stubject cop Add Stubject cop Aug 7, 2016
@backus
Copy link
Collaborator Author

backus commented Aug 7, 2016

I think this is actually pretty stable now. Running this against a lot of local projects yielded good results.

@backus backus force-pushed the feature/Stubject branch 3 times, most recently from ecb6c3e to 162b662 Compare August 7, 2016 04:09
@backus backus changed the title Add Stubject cop Add SubjectStub cop Aug 7, 2016
@backus backus force-pushed the feature/Stubject branch from fbaa79f to 9b361be Compare August 7, 2016 04:11
@backus backus force-pushed the feature/Stubject branch from 3ffdec6 to 5d76715 Compare August 7, 2016 04:14
@backus backus force-pushed the feature/Stubject branch from 5df089a to 4953398 Compare August 7, 2016 04:30
@backus backus merged commit 0f01f37 into master Aug 7, 2016
@backus backus deleted the feature/Stubject branch August 7, 2016 04:36
@andyw8
Copy link
Collaborator

andyw8 commented Jan 28, 2017

@turino As this PR is already merged, would you mind creating an issue for these enhancements? I agree they'd both be useful.

@turino
Copy link

turino commented Jan 28, 2017

@andyw8 I'd be happy to

markburns pushed a commit to markburns/rubocop-rails-ddd that referenced this pull request Nov 7, 2017
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