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 AnnotateRoutes::AnnotationProcessor and AnnotateRoutes::RemovalProcessor #793

Conversation

nard-tech
Copy link
Collaborator

cf. #790, #792

In order to refactor AnnoateRoutes, I added AnnotateRoutes::AnnotationProcessor and AnnotateRoutes::RemovalProcessor.

Refactoring AnnotateRoutes is finished in this PR and I would like to refactor AnnotateModels in a like way.

@ctran
Copy link
Owner

ctran commented Mar 24, 2021

Hello @nard-tech,

Is this still valid? If so, can you fix the conflict?

@ctran ctran added the reviewed label Mar 24, 2021
@nard-tech
Copy link
Collaborator Author

Hello @ctran, the conflict is in .rubocop_todo.yml. So I excuted rubocop --auto-gen-config and re-generate .rubocop_todo.yml.

Now this PR is valid. Please merge if there is no problem.

@nard-tech nard-tech force-pushed the feature/refactor_annotate_routes/processors branch from 2cf8313 to 4697388 Compare March 29, 2021 14:01
This was referenced Mar 31, 2021
@nard-tech
Copy link
Collaborator Author

nard-tech commented Apr 7, 2021

@ctran I cherry-picked the commit in #865 (e874241) to this pull request, and the CI worked fine.

So after #865 merged, I will rebase this pull request onto develop branch.

Please review again after rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants