Skip to content
This repository was archived by the owner on Dec 9, 2024. It is now read-only.

Conversation

@tbarlow12
Copy link
Contributor

@tbarlow12 tbarlow12 requested review from mydiemho, pjlittle and wbreza May 23, 2019 18:53
Copy link
Contributor

@mydiemho mydiemho left a comment

Choose a reason for hiding this comment

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

the script from VoTT was a little funky requiring v-prefix for source and not dest (or vice versa I can't remember). We should change it here to be more consistent.

@pjlittle
Copy link
Contributor

Yeah, this is probably a good opportunity to revisit the workflow and look at using npm version to drive a little more consistently. I think the script is a fine place to start. We haven't talked much about branches/tags/releases, in the context of this project. What we used on VoTT feels like it will work well here.

@pjlittle
Copy link
Contributor

Let's chat collectively to iron out expected release workflows.

@tbarlow12
Copy link
Contributor Author

@pjlittle sounds good. We can hold off on this for now. We also should talk about how often we merge dev into master

@tbarlow12 tbarlow12 closed this May 29, 2019
@tbarlow12 tbarlow12 deleted the tabarlow/changelog branch June 7, 2019 17:27
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.

5 participants