Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Adding strict typing for PHP classes in images doc#9587

Merged
pamil merged 1 commit into
Sylius:1.1from
Roshyo:patch-1
Jul 13, 2018
Merged

Adding strict typing for PHP classes in images doc#9587
pamil merged 1 commit into
Sylius:1.1from
Roshyo:patch-1

Conversation

@Roshyo
Copy link
Copy Markdown
Contributor

@Roshyo Roshyo commented Jul 13, 2018

This part of the doc was lacking strict PHP typing.

Q A
Branch? 1.0, 1.1, 1.2
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

Adding strict typing for the cookbook in images.

@Roshyo Roshyo requested a review from CoderMaggie as a code owner July 13, 2018 06:07
@Zales0123 Zales0123 added the Documentation Documentation related issues and PRs - requests, fixes, proposals. label Jul 13, 2018
@CoderMaggie
Copy link
Copy Markdown
Member

Hi there @Roshyo, nice to see you in our community as a contributor 👋 💪!

One thing that we need to change here is the base branch of this PR. It should be opened to the 1.0 branch as it is a fix for an already existing doc, working since the 1.0 version (am I right @pamil ?).
Could you please rebase with the 1.0 branch?

@Roshyo
Copy link
Copy Markdown
Contributor Author

Roshyo commented Jul 13, 2018

@CoderMaggie Hi,

I'm ok with that, but since I'm definitely not familiar with GitHub (boooh shame on me) I'm not sure about how to do that. Can you help me ?

@pamil
Copy link
Copy Markdown
Contributor

pamil commented Jul 13, 2018

@CoderMaggie 1.0 is for security fixes only right now, 1.1 is for bug fixes and enhancements like this.

@Roshyo I'll do it for you :)

@pamil pamil changed the base branch from master to 1.1 July 13, 2018 08:22
This part of the doc was lacking strict PHP typing.
@pamil pamil merged commit 360a406 into Sylius:1.1 Jul 13, 2018
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Jul 13, 2018

Thank you, Stephane! 🥇

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

Labels

Documentation Documentation related issues and PRs - requests, fixes, proposals.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants