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

Fixed instantiating ViewController basing on StoryboardSceneBased #38

Merged
merged 2 commits into from Dec 5, 2016
Merged

Fixed instantiating ViewController basing on StoryboardSceneBased #38

merged 2 commits into from Dec 5, 2016

Conversation

ghost
Copy link

@ghost ghost commented Nov 29, 2016

No description provided.

Copy link
Owner

@AliSoftware AliSoftware left a comment

Choose a reason for hiding this comment

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

Could you please add an entry in the CHANGELOG to mention the fix and credit yourself? Thanks!

* Fix `instantiate()` implementation on `StoryboardSceneBased` ViewControllers.  
  [@jakubgert](https://github.com/jakubgert)
  [#38](https://github.com/AliSoftware/Reusable/pull/38)

@ghost
Copy link
Author

ghost commented Nov 29, 2016

Sure. Should I bump version in CHANGELOG to e.g. 3.0.1 or you will do it?

@AliSoftware
Copy link
Owner

I usually ask people to add their changelog entry under a # Unreleased or # Master section title if it doesn't exist already, and change that title to the appropriate version (respecting SemVer depending on the changes that happened since last release) myself when I decide to release…

But that being said… tbh I think I'll do the release immediately after I merge this PR (cause this is a pretty big and blocking bug), so you can immediately name that new section # 3.0.1 in your case 👍

Added description about fix
Bumped version
@AliSoftware
Copy link
Owner

🤔 I've had to restart the Travis-CI build 3 times because each time it errors (it doesn't launch the job and fail, no, it just errors without even launching the build…). Let's hope this time it gets better.

I have no idea if that's linked to the shortage that Travis-CI is currently experiencing with OSX jobs or not, so if we should change something to make it build again or not, I'm a bit confused…

(we have the same problem with #37)

@AliSoftware AliSoftware merged commit 6429481 into AliSoftware:master Dec 5, 2016
@AliSoftware
Copy link
Owner

Ok, I bail, I have no f* idea why Travis errors without even starting the build. Merging anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants