Skip to content

Conversation

@mz026
Copy link
Contributor

@mz026 mz026 commented Jan 7, 2016

No description provided.

Copy link
Collaborator

Choose a reason for hiding this comment

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

why do write it like this?

# pg\_restore

it has to be

# pg_restore

@igorshubovych
Copy link
Collaborator

`pg_restore -h {{localhost}} -p {{5432}} -U {{username}} -d {{db_name}} {{archive_file.dump}}`

This is complicated example, and you introduce 5 options in it. It is against tldr guidelines. Please split it.

Take a look at mysql page or

@mz026
Copy link
Contributor Author

mz026 commented Jan 7, 2016

just updated, thanks for reviewing!!

2016-01-07 21:15 GMT+08:00 Igor Shubovych [email protected]:

pg_restore -h {{localhost}} -p {{5432}} -U {{username}} -d {{db_name}} {{archive_file.dump}}

This is complicated example, and you introduce 5 options in it. It is
against tldr guidelines. Please split it.

Take a look at mysql page
https://github.com/tldr-pages/tldr/blob/master/pages/common/mysql.md or


Reply to this email directly or view it on GitHub
#630 (comment).

@igorshubovych
Copy link
Collaborator

I like it much more! Thanks. LGTM.

@igorshubovych
Copy link
Collaborator

@mz026 Could you plz squash the commits?

@rubenvereecken rubenvereecken added the new command Issues requesting creation of a new page or PRs adding a new page for a command. label Jan 7, 2016
@mz026 mz026 force-pushed the add-pg_restore-page branch from 13ffbad to 19a9bf7 Compare January 8, 2016 01:50
@mz026
Copy link
Contributor Author

mz026 commented Jan 8, 2016

squashed, thanks! @igorshubovych

Copy link
Member

Choose a reason for hiding this comment

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

existed --> existing

@notpeter
Copy link
Contributor

notpeter commented Jan 8, 2016

You wanna do one for pg_dump too?

@mz026
Copy link
Contributor Author

mz026 commented Jan 8, 2016

sure! I added a pg_dump page. @notpeter
Should I separate them into 2 PRs?

@notpeter
Copy link
Contributor

notpeter commented Jan 8, 2016

I'm not the repo owner but I'm sure one PR for paired commands like pg_dump & pg_restore is fine.

On Jan 7, 2016, at 10:16 PM, Yang-Hsing Lin [email protected] wrote:

sure! I added a pg_dump page. @notpeter
Should I separate them into 2 PRs?


Reply to this email directly or view it on GitHub.

@mz026
Copy link
Contributor Author

mz026 commented Jan 8, 2016

haha! I'll separate them then.

@mz026 mz026 force-pushed the add-pg_restore-page branch 2 times, most recently from 07518bc to 377fb3b Compare January 8, 2016 06:50
@waldyrious
Copy link
Member

@mz026 I get the feeling you misunderstood @notpeter. Indeed a single PR for related commands is quite acceptable especially since it's desirable to keep any suggested changes in sync. See for instance #605.

@mz026 mz026 force-pushed the add-pg_restore-page branch from 377fb3b to 999b868 Compare January 8, 2016 14:05
@mz026
Copy link
Contributor Author

mz026 commented Jan 8, 2016

thanks for clarifying @waldyrious . I added a pg_dump page on it 😄 @notpeter

@mz026 mz026 changed the title pg_restore: add page pg_restore/pg_dump: add page Jan 10, 2016
@igorshubovych
Copy link
Collaborator

It is almost good.
Could you please tune markup a little bit according to our recent changes in guidelines. Capital letters in the beginning and colons in the end of examples' descriptions.

@mz026 mz026 force-pushed the add-pg_restore-page branch from 999b868 to f4aebf6 Compare January 11, 2016 15:39
@mz026
Copy link
Contributor Author

mz026 commented Jan 11, 2016

Just updated, thanks! @igorshubovych

@igorshubovych
Copy link
Collaborator

Thank you for contribution! 👍

igorshubovych added a commit that referenced this pull request Jan 11, 2016
@igorshubovych igorshubovych merged commit efbceaf into tldr-pages:master Jan 11, 2016
@waldyrious waldyrious changed the title pg_restore/pg_dump: add page pg_restore, pg_dump: add pages Sep 22, 2016
@dmmqz dmmqz mentioned this pull request Sep 7, 2025
35 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new command Issues requesting creation of a new page or PRs adding a new page for a command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants