-
Notifications
You must be signed in to change notification settings - Fork 987
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
Autoload ProxyAPI files from /app/services #8993
Conversation
Issues: #34138 |
6b0a21f
to
82cf965
Compare
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.
Did you change the boot_server
just because you've seen it?
I know it is hard to resist this, but might be better unless we have incentive to do so.
Don't get me wrong, I like it, I don't agree with the initial reason for it (the one I suspect), it is just it might break stuff unnecessarily :)
82cf965
to
5ebeb4d
Compare
I was trying to make rubocop happy :D But if I think about it it would be safer to just update Anyway PR rebased & updated, ready for review |
5ebeb4d
to
8374ef2
Compare
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.
Thanks @stejskalleos ! This feels safer :)
No description provided.