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

Remove Debugging Guide#8538

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:masterfromweaverryan:new-no-debugging
Nov 10, 2017
Merged

Remove Debugging Guide#8538

weaverryan merged 1 commit intosymfony:masterfromweaverryan:new-no-debugging
Nov 10, 2017

Conversation

@weaverryan
Copy link
Member

In 3.4, the debugging article has details aboutbootstrap.php.cache, etc. But in 4 (as you can see), it has almost nothing! I think we should remove it (and remove the Guide link on Symfony.com for version 4). The individualdebug: commands should be documented in those sections - nobody will click "Debug" so they can learn about how to debug something specific (e.g. translations).

Cheers!

Copy link
Member

@javiereguiluzjaviereguiluz left a comment

Choose a reason for hiding this comment

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

I like this proposal. I'd be more happy if the debugging commands were already moved to their new locations instead of "losing" them ... but we can do that in a separate PR.

@weaverryan
Copy link
MemberAuthor

@javiereguiluz when we merge this, will the guide link automatically disappear on symfony.com (for 4.0 only)? Or does more work need to be done for that?

@wouterj
Copy link
Member

#7071 😢

@weaverryan
Copy link
MemberAuthor

@wouterj Oh no!!! I completely ran over your work :/. I'm sorry. My vote is still to merge this PR... because (as nice as your article is), it doesn't contain alot of content (I like the details ondump(), but we should show that elsewhere, and I also have seen that almost everyone knows about this function... somehow).

@javiereguiluzjaviereguiluz mentioned this pull requestOct 29, 2017
12 tasks
@weaverryan
Copy link
MemberAuthor

@javiereguiluz if you still agree with this proposal, could you remove the Debugging entry from symfony.com? Thanks for removing the other sections on my other PR :)

@weaverryan
Copy link
MemberAuthor

ping@javiereguiluz!

This is not super important, but I'd love to finish it :)

@javiereguiluz
Copy link
Member

I've removed the link to "Debug" ... it will be online after the next deploy. Thanks!

@weaverryanweaverryan merged commitacdc22a intosymfony:masterNov 10, 2017
weaverryan added a commit that referenced this pull requestNov 10, 2017
This PR was merged into the master branch.Discussion----------Remove Debugging GuideIn 3.4, the debugging article has details about `bootstrap.php.cache`, etc. But in 4 (as you can see), it has almost nothing! I think we should remove it (and remove the Guide link on Symfony.com for version 4). The individual `debug:` commands should be documented in those sections - nobody will click "Debug" so they can learn about how to debug something specific (e.g. translations).Cheers!Commits-------acdc22a no more debugging
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@javiereguiluzjaviereguiluzjaviereguiluz approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@weaverryan@wouterj@javiereguiluz@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp