search

cobbler / cobbler-web

Our new seperate Angular based web interface for Cobbler
https://cobbler.github.io/cobbler-web/
MIT License
13 stars 7 forks source link

Remove `app.module.ts` #265

Open SchoolGuy opened 2 months ago

SchoolGuy commented 2 months ago

Is your feature request related to a problem? Please describe. As a developer I don't want to have dead code in the repository.

Describe the solution you'd like Remove the app module without the application breaking. This shouldn't be the case since we transitioned to Angular 17 but it should be ensured.

Describe alternatives you've considered None

Additional context None

Subhraaa commented 1 month ago

why would you remove app.module.ts? please elaborate

SchoolGuy commented 1 month ago

@Subhraaa Because with Angular 17+ standalone components were added to Angular and if the whole project is ported to them then it is not needed anymore. Since this project was ported to Angular Standalone the file remained in the repository but it is now time to remove it.