-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[ADR] Declaring services as public in container #13641
Conversation
lchrusciel
commented
Feb 11, 2022
Q | A |
---|---|
Branch? | 1.10 |
Bug fix? | no |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Related tickets | |
License | MIT |
e4313d3
to
4e9ad36
Compare
What about performances? Isn't it heavier to have all services public? |
According to Mateusz posts(#13641 (comment)) we already have 964 services, all of them(or at least majority) are public. So creation of new private services probably have neglectable influence over the website. |
4e9ad36
to
2811f19
Compare
The base of this pull-request was changed, you need fetch and reset your local branch Unless you added new commits (to this branch) locally that you did not push yet, Feel free to ask for assistance when you get stuck 👍 |
2811f19
to
6a4313d
Compare
6a4313d
to
6adb39d
Compare
Thanks, Łukasz! 🎉 |