Skip to content

Conversation

@genebean
Copy link
Contributor

@genebean genebean commented Jul 2, 2020

This chane updates docs to reflect that ownership has moved to the DIO team at Puppet. It also includes some formatting changes.

vmfloaty.gemspec Outdated
s.version = Vmfloaty::VERSION
s.authors = ['Brian Cain']
s.email = ['brianccain@gmail.com']
s.authors = ['Puppet']
Copy link
Contributor

Choose a reason for hiding this comment

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

Hey! I'd prefer to still be the author here since I wrote it :p Feel free to update the email though to include puppet, but I'd rather not erase the fact that I authored this project 😄

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree, Brian wrote this and we should continue to credit him in the gemspec. Please update this to an array with both Brian and Puppet, @genebean.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will do

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you @genebean 🙏

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I’ll push an update as soon as my internet gets unbusted

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This should be good now

vmfloaty.gemspec Outdated
s.authors = ['Brian Cain']
s.email = ['brianccain@gmail.com']
s.authors = ['Puppet']
s.email = ['dio-gems@puppet.com']
Copy link
Contributor

@briancain briancain Jul 2, 2020

Choose a reason for hiding this comment

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

Suggested change
s.email = ['dio-gems@puppet.com']
s.email = ['brianccain@gmail.com', 'dio-gems@puppet.com']

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This should also be good now

This chane updates docs to reflect that ownership has moved to the DIO
team at Puppet. It also includes some formatting changes.
@genebean
Copy link
Contributor Author

genebean commented Jul 2, 2020

@briancain & @highb I think I have made all requested changes and then some :)

@highb
Copy link
Contributor

highb commented Jul 2, 2020

Aw, nice idea on the "Special Thanks" section. 😄 This looks good to me; @briancain any additional feedback?

@briancain
Copy link
Contributor

Yeah! Looks good to me. Thanks for the special thanks section ❤️

@highb highb merged commit 2de284f into master Jul 2, 2020
@highb highb deleted the docs-update branch July 2, 2020 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants