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

Upgraded to Castle.Core 4.2.0 to 4.3.1. #413

Merged
3 commits merged into from Jun 22, 2018
Merged

Upgraded to Castle.Core 4.2.0 to 4.3.1. #413

3 commits merged into from Jun 22, 2018

Conversation

ghost
Copy link

@ghost ghost commented Jun 21, 2018

@ghost ghost requested review from stakx and jonorossi June 22, 2018 00:01
CHANGELOG.md Outdated
@@ -3,6 +3,7 @@
## Unreleased

Enhancements:
- Upgraded to Castle.Core 4.2.0 to 4.3.1. https://github.com/castleproject/Core/releases/tag/v4.3.1 (@fir3pho3nixx).
Copy link
Member

Choose a reason for hiding this comment

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

Maybe just update the line below 😉:

Copy link
Member

@stakx stakx Jun 22, 2018

Choose a reason for hiding this comment

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

Small nit (in response to @jonorossi's comment on this), I guess it should read #413 (this PR) instead of #409 (a previous PR).

Copy link
Member

@stakx stakx left a comment

Choose a reason for hiding this comment

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

Apart from the changelog entry, which looks a little funky right now, looks good to me!

CHANGELOG.md Outdated
@@ -3,6 +3,7 @@
## Unreleased

Enhancements:
- Upgraded to Castle.Core 4.2.0 to 4.3.1. https://github.com/castleproject/Core/releases/tag/v4.3.1 (@fir3pho3nixx).
Copy link
Member

@stakx stakx Jun 22, 2018

Choose a reason for hiding this comment

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

Small nit (in response to @jonorossi's comment on this), I guess it should read #413 (this PR) instead of #409 (a previous PR).

@ghost
Copy link
Author

ghost commented Jun 22, 2018

Thanks for checking, did this at 02:00 in the morning. :)

@ghost ghost merged commit 35ebd6e into castleproject:master Jun 22, 2018
@jonorossi
Copy link
Member

Did I miss something? The changelog still has 2 entries for the same release:

Enhancements:
- Upgraded to Castle.Core 4.2.0 to 4.3.1 (@fir3pho3nixx, #413)
...
- Upgraded Castle.Core from 4.2.0 to 4.3.0 (@fir3pho3nixx, #409)

@ghost
Copy link
Author

ghost commented Jun 26, 2018

Will fix.

This pull request was closed.
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