Adding Plume install steps for Yunohost #15

Merged
yalh76 merged 6 commits from plume/documentation:master into master 5 years ago
yalh76 commented 5 years ago (Migrated from github.com)
There is no content yet.
igalic (Migrated from github.com) requested changes 5 years ago
igalic (Migrated from github.com) left a comment

👀

👀
igalic (Migrated from github.com) commented 5 years ago

"you need" might sound a bit better here.?

"you need" might sound a bit better here.?
igalic (Migrated from github.com) commented 5 years ago

please add a newline here for markdown processors which are not Github…

please add a newline here for markdown processors which are not Github…
igalic (Migrated from github.com) commented 5 years ago

i would make this two bash sections, and leave the "or" out of it.

i would make this **two** `bash` sections, and leave the "or" out of it.
igalic (Migrated from github.com) commented 5 years ago

perhaps briefly explain what the difference is between the two methods:

"or, if you prefer to go bleeding edge,"

perhaps briefly explain what the difference is between the two methods: "or, if you prefer to go bleeding edge,"
yalh76 (Migrated from github.com) reviewed 5 years ago
yalh76 (Migrated from github.com) commented 5 years ago

Yes without problem, made the change, my english is not subtle enough ^^

Yes without problem, made the change, my english is not subtle enough ^^
yalh76 (Migrated from github.com) reviewed 5 years ago
yalh76 (Migrated from github.com) commented 5 years ago

I removed the first way to install using SSH as the prerequisite is to add the community app list in Yunohost. But in a few weeks, community and official lists will be merged in a single list. So I let the universal way to install it.

I removed the first way to install using SSH as the prerequisite is to add the community app list in Yunohost. But in a few weeks, community and official lists will be merged in a single list. So I let the universal way to install it.
igalic (Migrated from github.com) requested changes 5 years ago
igalic (Migrated from github.com) left a comment

spelling

spelling
igalic (Migrated from github.com) commented 5 years ago

many people might be very confused with the rather rare spelling of "connexion"

please use "connection"

many people might be very confused with the rather rare spelling of "connexion" please use "connection"
yalh76 (Migrated from github.com) reviewed 5 years ago
yalh76 (Migrated from github.com) commented 5 years ago

My bad, usual french mistake

My bad, usual french mistake
igalic (Migrated from github.com) approved these changes 5 years ago
igalic (Migrated from github.com) left a comment

👍

👍
igalic commented 5 years ago (Migrated from github.com)

thanks @yalh76

thanks @yalh76

Reviewers

The pull request has been merged as 8c7299dcad.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b master master
git pull origin master

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff master
git push origin master
Sign in to join this conversation.
Loading…
There is no content yet.