#4780 Update README to make commands in Docker Compose clearer
Merged by pingou. Opened by deveshks.
deveshks/pagure update-readme-make-docker-commands-clearer  into  master

Download 4780.patch

Signed-off-by: Devesh Kumar Singh deveshkusingh@gmail.com
Fixes: https://pagure.io/pagure/issue/4779

in order to populate the container with test data and create a new account, run -> run this command to populate the container with test data and create a new account

in order to populate the container with test data and create a new account, run -> run this command to populate the container with test data and create a new account

This wording is more clearer than mine. Thanks, I will make the change

1 new commit added

  • Fixed wording for docker command which creates new users

@deveshks Could you please squash into one commit?

You can do so by doing the following:

$ git rebase -i HEAD~2
<change the new commit to 'fixup' from 'pick' and save+quit>
$ git push -f

rebased onto cb116e3fdd779340520f169a401f720ac913108e

Hi @ngompa ,

I have squashed the two commits in one.

:thumbsup:

Hi @ngompa,

Can I now merge it myself, or perhaps you can merge it if it looks good.

We tend to wrap lines at around 80 characters, including in the README.

I'll merge this PR manually so that I can do the wrapping before pushing and you won't have to bother with it :)

Thanks for the patch!

Commit 4e6f3876 fixes this pull-request

Pull-Request has been merged by pingou

Hi @pingou,

Thanks for the comment and the suggestions. I will keep it in mind in future PRs

Metadata