Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

clarify the routing component documentation a bit#6023

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
weaverryan merged 1 commit intosymfony:2.3fromdbu:clarify-routing-component
Jan 10, 2016
Merged

clarify the routing component documentation a bit#6023

weaverryan merged 1 commit intosymfony:2.3fromdbu:clarify-routing-component
Jan 10, 2016

Conversation

@dbu
Copy link
Contributor

@dbudbu commentedDec 16, 2015

QA
Doc fix?yes
New docs?no
Applies to2.3+
Fixed ticketsRelates to#5709

@dbudbu mentioned this pull requestDec 16, 2015
5 tasks
Copy link
Member

Choose a reason for hiding this comment

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

missing angle bracket

Copy link
Member

Choose a reason for hiding this comment

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

and double backticks should be single ones

@dbu
Copy link
ContributorAuthor

dbu commentedDec 18, 2015

fixed

@xabbuh
Copy link
Member

👍

@dbu
Copy link
ContributorAuthor

dbu commentedJan 4, 2016

anything left to do here or can this be merged?

@xabbuh
Copy link
Member

@javiereguiluz@weaverryan@wouterj What do you think?

Copy link
Member

Choose a reason for hiding this comment

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

I'd removeduring match because it looks redundant.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

fixed

@weaverryan
Copy link
Member

Nice job@dbu, thanks :)

@weaverryanweaverryan merged commit940fb8f intosymfony:2.3Jan 10, 2016
weaverryan added a commit that referenced this pull requestJan 10, 2016
This PR was merged into the 2.3 branch.Discussion----------clarify the routing component documentation a bit| Q             | A| ------------- | ---| Doc fix?      | yes| New docs?     | no| Applies to    | 2.3+| Fixed tickets | Relates to#5709Commits-------940fb8f clarify the routing component documentation a bit
weaverryan added a commit that referenced this pull requestJan 10, 2016
@dbudbu deleted the clarify-routing-component branchOctober 18, 2018 17:26
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dbu@xabbuh@weaverryan@javiereguiluz@wouterj

[8]ページ先頭

©2009-2025 Movatter.jp