8000 Dont advertize what symfony/symfony "provides" by nicolas-grekas · Pull Request #29965 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

Dont advertize what symfony/symfony "provides" #29965

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
Jan 24, 2019

Conversation

nicolas-grekas
Copy link
Member
Q A
Branch? 3.4
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

No need to maintain not advertize this IMHO.

@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Jan 23, 2019
@fabpot
Copy link
Member
fabpot commented Jan 23, 2019

+1 for master, not sure we can do it on 3.4

@nicolas-grekas nicolas-grekas changed the base branch from 3.4 to master January 24, 2019 17:52
@nicolas-grekas
Copy link
Member Author

Updated for master.

@chalasr chalasr modified the milestones: 3.4, next Jan 24, 2019
@nicolas-grekas nicolas-grekas merged commit e8a7a0e into symfony:master Jan 24, 2019
nicolas-grekas added a commit that referenced this pull request Jan 24, 2019
…grekas)

This PR was merged into the 4.3-dev branch.

Discussion
----------

Dont advertize what symfony/symfony "provides"

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

No need to maintain not advertize this IMHO.

Commits
-------

e8a7a0e Dont advertize what symfony/symfony "provides"
@nicolas-grekas nicolas-grekas deleted the no-root-provide branch January 24, 2019 22:23
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.3 Apr 30, 2019
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.

4 participants
0