[!!!][TASK] Lock provider type in template artifact

[eliashaeussler]

Restoring a build artifact cannot be done properly if the original template provider cannot be determined. Hence, we now use the provider type instead of its name.

This is breaking change as it results in a new build artifact version (1679497137 → 1688738958).

[codeclimate]

Code Climate has analyzed commit 93581d5 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 95.7% (0.0% change).

View more on Code Climate.

[codecov]

Codecov Report

Merging #95 (cd11f37) into 3.x (5019fc6) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                3.x      #95      +/-   ##
============================================
+ Coverage     98.20%   98.29%   +0.09%     
- Complexity      770      776       +6     
============================================
  Files           111      112       +1     
  Lines          2445     2462      +17     
============================================
+ Hits           2401     2420      +19     
+ Misses           44       42       -2     

Impacted Files	Coverage Δ
src/Builder/Artifact/TemplateArtifact.php	100.00% <ø> (ø)
...Builder/Artifact/Migration/Migration1688738958.php	100.00% <100.00%> (ø)
src/Builder/ArtifactGenerator.php	100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[mteu]

This seems ready for shipment. What could possibly be open here?

[eliashaeussler]

This seems ready for shipment. What could possibly be open here?

@mteu Once #96 is merged, I'll change this PR to lock the provider type instead of class. That's a little bit more straightforward in terms of artifact reconstitution.

[mteu]

This seems ready for shipment. What could possibly be open here?

@mteu Once #96 is merged, I'll change this PR to lock the provider type instead of class. That's a little bit more straightforward in terms of artifact reconstitution.

Makes sense, thank you.

[mteu]

@eliashaeussler 👍

Wanna wait for codeclimate or fire at once?

[mteu]

👏

[eliashaeussler]

@eliashaeussler 👍

Wanna wait for codeclimate or fire at once?

Thanks for your approval. I'm too tired to wait for all those obviously unstable CGL tools.

[mteu]

Thanks for your approval. I'm too tired to wait for all those obviously unstable CGL tools.

I feel you.