-
-
Notifications
You must be signed in to change notification settings - Fork 589
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
[Common] Refactor Services #212
Conversation
f7cee40
to
ec0fab1
Compare
Services are Refactored and Unittests should be passing now. Unit-Tests for more than the first Service require a very significant addon to the testing system, hence are delayed untill a later date. TODO:
|
9744dfb
to
2e909e5
Compare
Should be mostly done, just needs some tests to finish and maybe some bugtracing. |
a4a3693
to
5c8ba43
Compare
- services (dict, including main service) - additionalServices (list)
5c8ba43
to
9f0ba8f
Compare
Awesome, thanks for the review @tprelog |
* Refactor All Services - services (dict, including main service) - additionalServices (list) * Add Documentation and standardised questions.yaml layout for services * Update all existing Apps to 2.0.0 * Fix whiteline error * fix addons
* Refactor All Services - services (dict, including main service) - additionalServices (list) * Add Documentation and standardised questions.yaml layout for services * Update all existing Apps to 2.0.0 * Fix whiteline error * fix addons
* Refactor All Services - services (dict, including main service) - additionalServices (list) * Add Documentation and standardised questions.yaml layout for services * Update all existing Apps to 2.0.0 * Fix whiteline error * fix addons
* Refactor All Services - services (dict, including main service) - additionalServices (list) * Add Documentation and standardised questions.yaml layout for services * Update all existing Apps to 2.0.0 * Fix whiteline error * fix addons
* Refactor All Services - services (dict, including main service) - additionalServices (list) * Add Documentation and standardised questions.yaml layout for services * Update all existing Apps to 2.0.0 * Fix whiteline error * fix addons
* Refactor All Services - services (dict, including main service) - additionalServices (list) * Add Documentation and standardised questions.yaml layout for services * Update all existing Apps to 2.0.0 * Fix whiteline error * fix addons
* Refactor All Services - services (dict, including main service) - additionalServices (list) * Add Documentation and standardised questions.yaml layout for services * Update all existing Apps to 2.0.0 * Fix whiteline error * fix addons
This PR is locked to prevent necro-posting on closed PRs. Please create a issue or contact staff on discord if you want to further discuss this |
Description
This PR refactors the services and breaks support with current code.
Hence it's merged into
Staging
notMaster
.Staging
will be merged when next TrueNAS SCALE release hitsIt aims to rework the services in a more sustainable fassion
Fixes #211
Type of change
How Has This Been Tested?
Notes:
Checklist: