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

Skip to content

[DX] Redirect to proper Symfony version documentation #27104

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

Merged
merged 1 commit into from
May 3, 2018

Conversation

noniagriconomie
Copy link
Contributor

@noniagriconomie noniagriconomie commented Apr 30, 2018

Q A
Branch? 3.4
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets none
License MIT
Doc PR

Just a small improvement
I think this can make some dev target the right documentation instead of reading an other version that the one they are developing on

EDIT: I do not know if including the constant directly is the good choice, or it is better to add a global symfony version collector?

@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Apr 30, 2018
@noniagriconomie noniagriconomie changed the title Redirect to proper Symfony version documentation [DX] Redirect to proper Symfony version documentation Apr 30, 2018
@noniagriconomie
Copy link
Contributor Author

@fabpot understood, target blank removed

@lyrixx
Copy link
Member

lyrixx commented May 2, 2018

The target branch is not good. It should be either 2.7 (bug fix) or master (new feature).
I don't know what is it though

@noniagriconomie
Copy link
Contributor Author

noniagriconomie commented May 2, 2018

This code does not exists on 2.x
And as 3.4 is an lts and it is code on dev only, i think it should be discussed/accepted IMO
What do you think?

@fabpot fabpot changed the base branch from 3.4 to master May 3, 2018 06:28
Copy link
Member

@fabpot fabpot left a comment

Choose a reason for hiding this comment

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

for master

@fabpot
Copy link
Member

fabpot commented May 3, 2018

Thank you @noniagriconomie.

@fabpot fabpot merged commit d611160 into symfony:master May 3, 2018
fabpot added a commit that referenced this pull request May 3, 2018
…(noniagriconomie)

This PR was squashed before being merged into the 4.1-dev branch (closes #27104).

Discussion
----------

[DX] Redirect to proper Symfony version documentation

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | none
| License       | MIT
| Doc PR        |

Just a small improvement
I think this can make some dev target the right documentation instead of reading an other version that the one they are developing on

EDIT: I do not know if including the constant directly is the good choice, or it is better to add a global symfony version collector?

Commits
-------

d611160 [DX] Redirect to proper Symfony version documentation
@noniagriconomie noniagriconomie deleted the patch-1 branch May 3, 2018 10:23
@fabpot fabpot mentioned this pull request May 7, 2018
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.

7 participants