docker / app

Make your Docker Compose applications reusable, and share them on Docker Hub
Apache License 2.0
1.57k stars 177 forks source link

Remove renderers #592

Closed rumpl closed 5 years ago

rumpl commented 5 years ago

- What I did

Removed the renderers from the code base.

Having different renderers (mustache, gotemplate, yatee) was an experimental feature we had prior to going down the CNAB route. We realized that this kind of templates is a wrong solution for docker app. We want to have a simpler, declarative file format that greatly simplifies app management/sharing.

- How I did it

With the key

- A picture of a cute animal (not mandatory but encouraged) source

codecov[bot] commented 5 years ago

Codecov Report

Merging #592 into master will increase coverage by 0.15%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #592      +/-   ##
==========================================
+ Coverage      71%   71.16%   +0.15%     
==========================================
  Files          55       53       -2     
  Lines        3242     2802     -440     
==========================================
- Hits         2302     1994     -308     
+ Misses        644      552      -92     
+ Partials      296      256      -40
Impacted Files Coverage Δ
internal/version.go 92.85% <ø> (-0.48%) :arrow_down:
render/render.go 76.38% <100%> (+4.78%) :arrow_up:
internal/compose/compose.go 74.35% <100%> (+3.62%) :arrow_up:
internal/packager/cnab.go 97.36% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 4e71bce...0d7c51e. Read the comment docs.