coolaj86/go-serviceman
2
2
Fork 0
Código Incidencias Pull Requests Lanzamientos Wiki Actividad

path option #7

Cerrada
ryanburnette desea fusionar 2 commits de (eliminado):path en master
recuperar de: (eliminado):path
fusionar en: coolaj86:master
Conversación 4 Commits 2 Archivos modificados 4 +34
ryanburnette comentado 2019-07-17 15:23:59 +00:00
Contribuyente por primera vez
Copiar enlace

Path option

  • add --path option to explicitly set path
  • update README.md

https://stackoverflow.com/questions/35439123/systemd-environment-directive-to-set-path
#6

Path option - [x] add `--path` option to explicitly set path - [x] update `README.md` https://stackoverflow.com/questions/35439123/systemd-environment-directive-to-set-path https://git.coolaj86.com/coolaj86/go-serviceman/issues/6
ryanburnette comentado 2019-07-17 16:05:32 +00:00
Autoría
Contribuyente por primera vez
Copiar enlace

Status

I'll know when I get back in a couple hours if the changes work. I believe before merging I should add comments and an example to README.md. @coolaj86 feedback?

Status I'll know when I get back in a couple hours if the changes work. I believe before merging I should add comments and an example to `README.md`. @coolaj86 feedback?
ryanburnette comentado 2019-07-18 13:58:49 +00:00
Autoría
Contribuyente por primera vez
Copiar enlace

@coolaj86 Take a look. If you're happy with the update to the lib and the readme then this is ready to merge.

I suggest doing a squashed merge because of my automatic commit messages.

@coolaj86 Take a look. If you're happy with the update to the lib and the readme then this is ready to merge. I suggest doing a squashed merge because of my automatic commit messages.
ryanburnette comentado 2019-08-04 01:39:54 +00:00
Autoría
Contribuyente por primera vez
Copiar enlace

@coolaj86 Thanks for helping me improve the PR today. We changed the --path option so it makes use of conf.Envs which was already implemented into the templates. That was a big improvement.

Ready to squash and merge.

@coolaj86 Thanks for helping me improve the PR today. We changed the `--path` option so it makes use of `conf.Envs` which was already implemented into the templates. That was a big improvement. Ready to squash and merge.
coolaj86 comentado 2019-08-05 10:49:32 +00:00
Propietario
Copiar enlace

Merged and published as v0.6.0

Once we get list and remove implemented I think I'll be happy to call this v1.0.0

Merged and published as v0.6.0 Once we get list and remove implemented I think I'll be happy to call this v1.0.0
coolaj86 cerró este pull request 2019-08-05 10:49:32 +00:00

Se cerró la solicitud de incorporación

Este pull request no se puede reabrir porque la rama fue eliminada.
Inicie sesión para unirse a esta conversación.
Revisores
No Reviewers
Etiquetas
No hay elementos
Sin etiquetas
Milestone
No hay elementos
Sin Milestone
Asignados
Limpiar asignados
coolaj86
No asignados
2 participantes
Notificaciones
Fecha de vencimiento
Sin fecha de vencimiento.
Dependencias

No se han establecido dependencias.

Referencia: coolaj86/go-serviceman#7
No se ha proporcionado una descripción.
Eliminar rama "(eliminado):path "

Eliminar una rama es permanente. Aunque la rama eliminada puede continuar existiendo durante un corto tiempo antes de que sea eliminada, en la mayoría de los casos NO PUEDE deshacerse. ¿Continuar?