Add Snapcraft metadata and install/maintenance hooks #666

Merged
RAOF merged 5 commits from add-snapcraft into master 2019-09-16 10:33:26 +00:00
5 changed files with 141 additions and 0 deletions

48
snap/hooks/configure vendored Normal file
View file

@ -0,0 +1,48 @@
#!/bin/sh
db_type="$(snapctl get db.type)"
db_url="$(snapctl get db.url)"
if [ "${db_type}" = "sqlite" ]
then
if [ -n "${db_url}" ]
then
echo "sqlite backend does not use db.url key"
exit 1
fi
elif [ "${db_type}" = "postgres" ]
then
if [ -z "${db_url}" ]
then
echo "postgres backend requires db.url to be set"
exit 1
fi
elif [ -n "${db_type}" ]
then
echo "Invalid db.type: " ${db_type}
exit 1
fi
base_url="$(snapctl get base-url)"
enabled="$(snapctl get enabled)"
if [ -n ${enabled} -a \( "${enabled}" != "true" -a "${enabled}" != "false" \) ]
then
echo "Invalid 'enabled' setting: ${enabled}. Valid values are 'true' or 'false'"
exit 1
fi
if [ -n ${base_url} -a \( -z "${enabled}" -o "${enabled}" = "false" \) ]
then
echo "All required configuration available."
echo "Plume can now be enabled by setting 'snap set plume enabled=true' and restarting the service \
with 'snap restart plume'"
fi
if [ "${enabled}" = "true" -a ! -e ${SNAP_COMMON}/initial-migrations-run ]
then
cd ${SNAP}
exec ./set-environment bin/plm migration run --path ${SNAP_DATA}
touch ${SNAP_COMMON}/initial-migrations-run
fi
exit 0

3
snap/hooks/install Executable file
View file

@ -0,0 +1,3 @@
#!/bin/sh
openssl rand -base64 32 > ${SNAP_COMMON}/rocket-secret-key

11
snap/hooks/post-refresh Normal file
View file

@ -0,0 +1,11 @@
#!/bin/sh
db_type=$(snapctl get db.type)
if [ -z "${db_type}" ]
then
exit 0
fi
cd ${SNAP}
exec ./set-environment bin/plm migration run --path ${SNAP_DATA}

29
snap/local/set-environment Executable file
View file

@ -0,0 +1,29 @@
#!/bin/sh
enabled="$(snapctl get enabled)"
if [ -z "${enabled}" -o "${enabled}" != "true" ]
then
echo "Plume not yet enabled"
exit 1
fi
export BASE_URL="$(snapctl get base-url)"
database_type="$(snapctl get db.type)"
if [ z"${database_type}" = z"sqlite" ]
then
export DATABASE_URL=${SNAP_DATA}/plume.db
export MIGRATION_DIR=migrations/sqlite
else
# Must be postgres, so must have set db.url
export DATABASE_URL="$(snapctl get db.url)"
export MIGRATION_DIRECTORY=migrations/postgres
fi
ROCKET_ADDRESS="$(snapctl get listen.address)"
ROCKET_PORT="$(snapctl get listen.port)"
export ROCKET_SECRET_KEY="$(cat ${SNAP_COMMON}/rocket-secret-key)"
export SEARCH_INDEX="${SNAP_DATA}/search_index"
cd ${SNAP}
exec $@

50
snap/snapcraft.yaml Normal file
View file

@ -0,0 +1,50 @@
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
name: plume
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
base: core18
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
version: '0.3.0' # just for humans, typically '1.2+git' or '1.3.2'
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
summary: Multi-user blogging platform, federated over ActivityPub
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
description: |
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
Plume is a federated blogging platform, featuring:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
* A blog-centric approach: you can create as much blogs as you want with your account, to keep your different publications separated.
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
* Media management: you can upload pictures to illustrate your articles, but also audio files if you host a podcast, and manage them all from Plume.
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
* Federation: Plume is part of a network of interconnected websites called the Fediverse. Each of these websites (often called instances) have their own rules and thematics, but they can all communicate with each other.
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
* Collaborative writing: invite other people to your blogs, and write articles together.
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
grade: devel # must be 'stable' to release into candidate/stable channels
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
confinement: strict
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
apps:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
plume:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
daemon: simple
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
command: set-environment bin/plume
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
plugs:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
- network
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
- network-bind
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
plm:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
command: set-environment bin/plm
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
parts:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
plume:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
plugin: rust
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
source: .
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
rust-revision: nightly-2019-03-23
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
build-packages:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
- libssl-dev
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
- pkg-config
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
- libsqlite3-dev
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
- gettext
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
after:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
- cargo-web
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
override-build: |
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
export PATH=$PATH:$SNAPCRAFT_PROJECT_DIR/../.cargo/bin
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
cargo web deploy -p plume-front --release
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
cargo install --force --no-default-features --features sqlite --path . --root ${SNAPCRAFT_PART_INSTALL}
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
cargo install --force --no-default-features --features sqlite --path plume-cli --root ${SNAPCRAFT_PART_INSTALL}
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
cp -a assets migrations static target translations ${SNAPCRAFT_PART_INSTALL}
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
cp snap/local/set-environment ${SNAPCRAFT_PART_INSTALL}
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
stage-packages:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
- openssl
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
- libsqlite3-0
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
cargo-web:
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
plugin: rust
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
source: https://github.com/koute/cargo-web.git
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!
source-tag: 0.6.26
igalic commented 2019-09-07 10:35:22 +00:00 (Migrated from github.com)
Review

what do we need / usr diesel-cli for?

what do we need / usr diesel-cli for?
RAOF commented 2019-09-07 10:55:42 +00:00 (Migrated from github.com)
Review

The compile plume instructions say to install diesel_cli. Is that not necessary?

The [compile plume](https://docs.joinplu.me/installation/with/source-code) instructions say to install diesel_cli. Is that not necessary?
igalic commented 2019-09-07 12:13:26 +00:00 (Migrated from github.com)
Review

i don't think it's necessary.

@fdb-hiroshima?

*i* don't think it's necessary. @fdb-hiroshima?
elegaanz commented 2019-09-07 13:52:05 +00:00 (Migrated from github.com)
Review

It is for the 0.3.0 tag, but not for master (we have plm migration run now).

It is for the 0.3.0 tag, but not for master (we have `plm migration run` now).
elegaanz commented 2019-09-07 13:54:40 +00:00 (Migrated from github.com)
Review

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.

Maybe you are already aware of that, but just to have reminder: this description should probably be updated.
RAOF commented 2019-09-16 06:18:34 +00:00 (Migrated from github.com)
Review

Removed

Removed
RAOF commented 2019-09-16 06:19:36 +00:00 (Migrated from github.com)
Review

Updated. Please give the verbiage a once-over!

Updated. Please give the verbiage a once-over!