-
Notifications
You must be signed in to change notification settings - Fork 9
Conversation
5736365
to
0427d7f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
Boa! Docs são importantes! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acho que esta feature poderia estar por trás de algo tipo feature flag.
awsRegion, ok := os.LookupEnv("DEPLOYER_AWS_REGION") | ||
if !ok { | ||
fmt.Printf("DEPLOYER_AWS_REGION must be defined") | ||
os.Exit(1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por que somente retornar false
ou coisa do tipo, para que esta feature seja opcional? Se eu não tiver interesse ou usar dynamodb não consigo usar o app mais?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
concordaria de primeira, mas pensando na demanda dele com a gente, não podemos dar essa opção por questão de uma pessoal mal intencionada poderia usar essa feature quando quisesse que não fosse logada.
f2f90f3
to
5695782
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
o/
Closes #12