Skip to content
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

Add support for mapstructure decode tag for reusability #81

Merged
merged 6 commits into from
May 28, 2020
Merged

Add support for mapstructure decode tag for reusability #81

merged 6 commits into from
May 28, 2020

Conversation

JRemitz
Copy link
Contributor

@JRemitz JRemitz commented Oct 1, 2019

I would like to leverage the client v3 API structs for Packer objects to the Nutanix API server. This would require adding support for mapstructure based on Packer's documentation to simplify the process of parsing the Packer config.

I currently have Packer PR 8175 open to add support for Nutanix - any help/insight to align projects would be greatly appreciated!

This includes changes from #80 along with corresponding mapstructure tags.

@marinsalinas
Copy link
Contributor

@JRemitz could you update with the latest code from master?

@JRemitz
Copy link
Contributor Author

JRemitz commented May 15, 2020

I can and will take a look at this in a bit.

@JRemitz
Copy link
Contributor Author

JRemitz commented May 15, 2020

@marinsalinas - I've updated however I see that the linter is failing on some of the long lines due to the extended tags. What's the best way to break those down?

@JRemitz
Copy link
Contributor Author

JRemitz commented May 27, 2020

@marinsalinas - apologies for the delay but finally got around to "fixing" this. Please take a look and let me know what you think. Thanks!

Copy link
Contributor

@marinsalinas marinsalinas left a comment

Choose a reason for hiding this comment

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

Just resolve conflicts with go.mod

@JRemitz
Copy link
Contributor Author

JRemitz commented May 28, 2020

Merged - how about now, @marinsalinas ?

@JRemitz JRemitz requested a review from marinsalinas May 28, 2020 17:55
@marinsalinas marinsalinas merged commit 4ea61c4 into nutanix:master May 28, 2020
yannickstruyf3 pushed a commit to yannickstruyf3/terraform-provider-nutanix that referenced this pull request Nov 9, 2021
Add support for mapstructure decode tag for reusability
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.

2 participants