Skip to content
This repository was archived by the owner on Aug 20, 2021. It is now read-only.

Remove profile dependency from implementation code generation - #80

Merged
anweiss merged 1 commit into
docker-archive:masterfrom
minhaj10p:impl-excel-param
Feb 7, 2019
Merged

Remove profile dependency from implementation code generation#80
anweiss merged 1 commit into
docker-archive:masterfrom
minhaj10p:impl-excel-param

Conversation

@minhaj10p

Copy link
Copy Markdown
Contributor

Description

  1. Populate ImplementsProfile section from control excel sheet
  2. Removes dependency from profile input as arguments

Comment thread templates/implementation.go Outdated
"html/template"
)

//GetImplementationTemplate gets implementation template for implementation go struct file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit

Suggested change
//GetImplementationTemplate gets implementation template for implementation go struct file
// GetImplementationTemplate gets implementation template for implementation go struct file

@anweiss

anweiss commented Feb 7, 2019

Copy link
Copy Markdown
Contributor

Looks good @minhaj10p. Please rebase and we can merge.

Comment thread impl/implementation.go

@anweiss anweiss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@anweiss
anweiss merged commit ec6032a into docker-archive:master Feb 7, 2019
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.

2 participants