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

Update hapijs/topo to 1.0.3 from 1.0.2 #2671

Merged
merged 1 commit into from Jul 29, 2015
Merged

Conversation

@devinivy
Copy link
Member

@devinivy devinivy commented Jul 29, 2015

Resolves #2657 and #2671. See failing test at #2642.

@hueniverse hueniverse self-assigned this Jul 29, 2015
@hueniverse hueniverse added this to the 8.8.1 milestone Jul 29, 2015
hueniverse pushed a commit that referenced this issue Jul 29, 2015
@hueniverse hueniverse merged commit 1bc1ed5 into hapijs:master Jul 29, 2015
1 check passed
@AdriVanHoudt
Copy link
Contributor

@AdriVanHoudt AdriVanHoudt commented Jul 30, 2015

can you rename this to be like the issue, makes for a better changelog ;) http://hapijs.com/updates

@mtharrison
Copy link
Contributor

@mtharrison mtharrison commented Jul 30, 2015

The changelog has picked it up from the issue. Is it normal practice to make both a PR and an issue for an update? We could get doubling up of plugin stuff in the changelog if so.

screen shot 2015-07-30 at 09 01 00

@AdriVanHoudt
Copy link
Contributor

@AdriVanHoudt AdriVanHoudt commented Jul 30, 2015

yeah but it didn't do it for the others

@devinivy devinivy changed the title Update topo to 1.0.3 Update hapijs/topo to 1.0.3 from 1.0.2 Jul 30, 2015
@lock
Copy link

@lock lock bot commented Jan 9, 2020

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants