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

fix(scripts): Ensure determine-pkg-versions outputs correct info #261

Conversation

traviskaufman
Copy link
Contributor

@traviskaufman traviskaufman commented Feb 6, 2017

  • Ensure new version output by determine-pkg-versions is correct
  • Ensure proper publishConfig flags are sent to UpdatedPackagesCollector

@traviskaufman
Copy link
Contributor Author

@yeelan0319 actually going to make a few more changes here, so feel free to hold off on the review until I update the PR 😄 thanks!

- Ensure new version output by determine-pkg-versions is correct
- Ensure proper publishConfig flags are sent to UpdatedPackagesCollector
@traviskaufman traviskaufman force-pushed the hotfix/ensure-determine-pkg-versions-outputs-correct-default-version branch from 1428761 to a384e66 Compare February 6, 2017 23:38
@traviskaufman
Copy link
Contributor Author

@yeelan0319 updated! Feel free to review

@traviskaufman traviskaufman changed the title fix(scripts): Ensure determine-pkg-versions outputs the correct version fix(scripts): Ensure determine-pkg-versions outputs correct info Feb 6, 2017
@yeelan0319
Copy link
Contributor

Cool. I will give it a look tomorrow morning 👍

Copy link
Contributor

@yeelan0319 yeelan0319 left a comment

Choose a reason for hiding this comment

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

💯 LGTM!

@traviskaufman
Copy link
Contributor Author

TravisCI seems to be hanging on this PR...last two builds passed and branch update was merged in cleanly so going to squash and merge.

@traviskaufman traviskaufman merged commit 1097e6f into master Feb 7, 2017
@traviskaufman traviskaufman deleted the hotfix/ensure-determine-pkg-versions-outputs-correct-default-version branch February 7, 2017 19:59
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.

None yet

2 participants