-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added common dependencies #95
Added common dependencies #95
Conversation
fce8f8c
to
c716f18
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @utsab818, Thank you for this PR.
The generatehcl2NixConf
and WriteConfig
should not be in init/init.go
.
We are already doing these under init/model.go
.
Please have a look at how we are passing the values over to the model file and generate common dependencies using that.
c716f18
to
90c6f7a
Compare
@rakshitgondwal Please check this. I checked it for my go project and it seems to be working fine. |
90c6f7a
to
6bfccbe
Compare
dab5874
to
3cc9c40
Compare
Signed-off-by: utsab818 <utsabsapkota4231@gmail.com>
61d50cf
to
247a0ac
Compare
Signed-off-by: utsab818 <utsabsapkota4231@gmail.com>
Thank you for your contribution @utsab818! |
Fixes #89