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

Rename configs with defintions to definitions-* #185

Open
vjik opened this issue Aug 28, 2021 · 3 comments
Open

Rename configs with defintions to definitions-* #185

vjik opened this issue Aug 28, 2021 · 3 comments

Comments

@vjik
Copy link
Member

vjik commented Aug 28, 2021

commondefinitions
webdefinitions-web
consoledefinitions-console

This is will be more consistently with events, bootstrap, providers, routes and etc.

@vjik vjik changed the title Rename configs with defintions to defintiions-* Rename configs with defintions to definitions-* Aug 28, 2021
@samdark
Copy link
Member

samdark commented Aug 30, 2021

Good thing is that the prefix will be the same so files will appear grouped. Bad thing is that it looks more complicated.

@vjik
Copy link
Member Author

vjik commented Aug 30, 2021

Seems, variant with definitions more simplify:

definitions/*.php
definitions-web/*.php
definitions-console/*.php
bootstrap.php
bootstrap-web.php
bootstrap-console.php
events.php
events-web
events-console.php
params.php
providers.php
providers-web.php
providers-console.php
routes-backend.php
routes.php

VS

common/*.php
console/*.php
web/*.php
bootstrap.php
bootstrap-web.php
bootstrap-console.php
events.php
events-web.php
events-console.php
params.php
providers.php
providers-web.php
providers-console.php
routes-backend.php
routes.php

@samdark
Copy link
Member

samdark commented Aug 31, 2021

Related to yiisoft/demo#345

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

No branches or pull requests

2 participants