Skip to content

[DSL] Remove target blocks#399

Merged
c-harding merged 3 commits intodevelopfrom
untargeted
Apr 28, 2020
Merged

[DSL] Remove target blocks#399
c-harding merged 3 commits intodevelopfrom
untargeted

Conversation

@c-harding
Copy link
Copy Markdown
Contributor

@c-harding c-harding commented Apr 28, 2020

These aren’t being used, and as we always generate OpenAPI specs, there isn’t really any need for them until we also generate code from the OpenAPI specs. At the moment they’re just a way of specifying a language name, which may be useful one day, so I’ll leave a PR in for their readdition (#400).

These aren’t being used, and as we always generate OpenAPI specs, there isn’t really any need for them until we also generate code from the OpenAPI specs. At the moment they’re just a way of specifying a language name, which may be useful one day, so I’ll leave a PR in for their readdition.
@c-harding c-harding requested a review from jaylees14 as a code owner April 28, 2020 14:35
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 28, 2020

Codecov Report

Merging #399 into develop will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #399      +/-   ##
===========================================
+ Coverage    98.93%   98.95%   +0.02%     
===========================================
  Files          137      137              
  Lines         3179     3167      -12     
  Branches        44       51       +7     
===========================================
- Hits          3145     3134      -11     
+ Misses          34       33       -1     
Impacted Files Coverage Δ
...in/scala/temple/DSL/semantics/ServiceRenamer.scala 100.00% <ø> (+3.70%) ⬆️
src/main/scala/temple/DSL/syntax/DSLRootItem.scala 100.00% <ø> (ø)
src/main/scala/temple/ast/Metadata.scala 100.00% <ø> (ø)
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 e9a4d8b...6ae1495. Read the comment docs.

@c-harding c-harding merged commit 7d15e69 into develop Apr 28, 2020
@jaylees14 jaylees14 deleted the untargeted branch April 28, 2020 18:09
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.

3 participants