Skip to content

[DSL] Allow specifying targets to generate in the DSL#400

Draft
c-harding wants to merge 3 commits intodevelopfrom
target
Draft

[DSL] Allow specifying targets to generate in the DSL#400
c-harding wants to merge 3 commits intodevelopfrom
target

Conversation

@c-harding
Copy link
Copy Markdown
Contributor

Revert #399, so the Templefile can specify the targets to generate. This is needed for running a generator on the OpenAPI spec.

@c-harding c-harding requested a review from jaylees14 as a code owner April 28, 2020 14:46
@c-harding c-harding marked this pull request as draft April 28, 2020 14:47
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 28, 2020

Codecov Report

Merging #400 into develop will decrease coverage by 0.02%.
The diff coverage is 94.11%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #400      +/-   ##
===========================================
- Coverage    98.95%   98.93%   -0.03%     
===========================================
  Files          137      137              
  Lines         3167     3179      +12     
  Branches        51       44       -7     
===========================================
+ Hits          3134     3145      +11     
- Misses          33       34       +1     
Impacted Files Coverage Δ
src/main/scala/temple/DSL/syntax/DSLRootItem.scala 100.00% <ø> (ø)
src/main/scala/temple/ast/Metadata.scala 100.00% <ø> (ø)
...in/scala/temple/DSL/semantics/ServiceRenamer.scala 96.29% <66.66%> (-3.71%) ⬇️
src/main/scala/temple/DSL/semantics/Analyzer.scala 100.00% <100.00%> (ø)
...rc/main/scala/temple/DSL/semantics/Validator.scala 100.00% <100.00%> (ø)
src/main/scala/temple/ast/Templefile.scala 100.00% <100.00%> (ø)

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 7d15e69...5e82c0f. Read the comment docs.

@c-harding c-harding changed the base branch from untargeted to develop April 28, 2020 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant