Skip to content
This repository has been archived by the owner on Apr 20, 2024. It is now read-only.

Added Breadcrumb tag #10

Merged
merged 5 commits into from
Jun 21, 2018
Merged

Added Breadcrumb tag #10

merged 5 commits into from
Jun 21, 2018

Conversation

martinlasek
Copy link
Contributor

@martinlasek martinlasek commented Jun 19, 2018

Do we want to have child tags that make not much sense without their parent tags to be in the same file?

For example #breadCrumbTag() {} is the parent and #breadCrumbItemTag() {} is the child and doesn't make that much sense to be used without its parent 馃槉

@codecov
Copy link

codecov bot commented Jun 19, 2018

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@         Coverage Diff          @@
##           master   #10   +/-   ##
====================================
  Coverage       0%    0%           
====================================
  Files           7     8    +1     
  Lines         215   263   +48     
====================================
- Misses        215   263   +48
Impacted Files Coverage 螖
Sources/Bootstrap/Bootstrap.swift 0% <0%> (酶) 猬嗭笍
Sources/Bootstrap/Tags/Breadcrumb.swift 0% <0%> (酶)

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 3c8a162...e5a6a8d. Read the comment docs.

}

public final class BreadCrumbItemTag: TagRenderer {
public func render(tag: TagContext) throws -> EventLoopFuture<TemplateData> {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Future

@steffendsommer steffendsommer merged commit 26053f9 into master Jun 21, 2018
@steffendsommer steffendsommer deleted the feature/breadcrumb branch June 21, 2018 13:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants