Skip to content

Upgraded to Castle.Core 4.2.0 to 4.3.1. #413

Merged
3 commits merged intomasterfrom
unknown repository
Jun 22, 2018
Merged

Upgraded to Castle.Core 4.2.0 to 4.3.1. #413
3 commits merged intomasterfrom
unknown repository

Conversation

@ghost
Copy link

@ghost ghost commented Jun 21, 2018

@ghost ghost requested review from jonorossi and stakx June 22, 2018 00:01
CHANGELOG.md Outdated
## 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
## 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.

2 participants