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

Documenting the Console Helpers#1999

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 7 commits intosymfony:2.0fromSgoettschkes:issue1811
Dec 26, 2012
Merged

Documenting the Console Helpers#1999

weaverryan merged 7 commits intosymfony:2.0fromSgoettschkes:issue1811
Dec 26, 2012

Conversation

@Sgoettschkes
Copy link
Contributor

As suggested in#1811, this PR includes documention on the two Console Helpers:

  • DialogHelper
  • FormatterHelper

The new documents are included in the index and checked for sphinx issues.

@wouterj
Copy link
Member

GitHub don't let me create a PR on your repo, but you should change this format issues:wouterj/symfony-docs@69ce6c9

@Sgoettschkes
Copy link
ContributorAuthor

@wouterj Done. I think to be able to make a PR you would need to add my repository as a new remote, pull my branch and make changes. Maybe there is a better way like giving you push access to my repository or you giving me push access.

@wouterj
Copy link
Member

@Sgoettschkes well, I added your repo as a remote and have based my commit on your branch. But in order to create a PR on github you should choose a branch in the dropdown list and it looks like Github doesn't refresh it if the main repo is forked by a new user. So your repo was not in the list of repo's to request a PR and I wasn't able to add you in there...

@wouterjwouterj mentioned this pull requestDec 14, 2012
@Sgoettschkes
Copy link
ContributorAuthor

I would love to see this merged so we can get started documenting the new ProgressHelper and new DialogHelper capabilities.

Any change requests for this PR?

@wouterj
Copy link
Member

@Sgoettschkes 👍, but be aware that the ProgressHelper is already documented (it is included in theintroduction chapter right now, it should be copied into a new file inside thehelper dir)

@weaverryan
Copy link
Member

Hi Sebastian!

Sorry for the delay on merging this - it's a really great PR! I've now merged it in and made only a few minor changes.

As@wouterj said, theProgressHelperis documented already, but I much prefer your approach of placing these helpers in their own documents, instead of inside on big document. Could you move theProgressHelper docs into the new format? I'll merge everything up to master now so that the new code is available there.

Thanks!

@wouterj
Copy link
Member

@Sgoettschkes are you going to document these new helpers/features? (otherwise I will d it)

@Sgoettschkes
Copy link
ContributorAuthor

As said I'll do it. I'll also take care of adding the new features as menttioned in#2022!

@SgoettschkesSgoettschkes deleted the issue1811 branchDecember 26, 2012 19:15
@wouterj
Copy link
Member

ok, great job! (almost all components are documented right now 🚴)

weaverryan added a commit that referenced this pull requestDec 26, 2012
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Sgoettschkes@wouterj@weaverryan

[8]ページ先頭

©2009-2025 Movatter.jp