Corrigido permissões de arquivos

[farribeiro]

Closes #212

[MarceloCajueiro]

Pode descrever melhor o que foi feito? Qual era a permissão e qual ficou? Por que foi feito?

[williamespindola]

Não entendi o que este PR resolve, esta permissão esta causando algum problema para a aplicação executar ou acessar algum arquivo?

[farribeiro]

Padronização das permissões de arquivos do projeto, há outras modificações para PADRONIZAR/CONFORMIDADE como encoding, eol e eof

Fora retirar espaços em branco no final de linha

[eberfreitas]

Sobre este tipo de contribuição, o que acham disso?

rails/rails#13771 (comment)

[farribeiro]

Se foi considerado um ruído... então melhor partir para outros tipo de padronização como formatter, lint ou mesmo um transpiller.

Eu vou instalar um PHP lint no vscode!

[williamespindola]

Aproveitando o ensejo. @farribeiro você esta causando este "ruido" em todos o projeto, nos prs, issues e discussão no slack. Entendemos sua vontade de contribuir, mas não temos o mesmo tempo que você para acompanhar suas solicitações e questionamento. Tenha calma que as coisas vão acontecer.
Pare e olhe por um minuto. Você comenta em absolutamente todas a issues, todos os PRs, todos os canais no slack. Como ja disse antes la no slack e falo aqui novamente pondere sua atitudes, se você continuar assim alguns contribuidores ou até mesmo a Portabilis pode desistir de continuar com o open source.
Acredito que não falo por mim, mas por todos os outros que estão convivendo com suas atitudes.

[farribeiro]

Eu apenas fiz um cross, para não ter que escrever a mesma mensagem a todos PRs, Além de não deixar e largada... Foi um arremate

[farribeiro]

A respeito desta PR não o considero um ruído e sim uma NÃO-CONFORMIDADE

[MarceloCajueiro]

@eberfreitas acho super válido isso. Pode criar uma issue pra gente documentar no doc de contribuição?

@farribeiro eu concordo com a visão do @williamespindola. Vá com calma, não adianta querer mudar tudo da noite para o dia, citar todo mundo, querer atenção e respostas imediatas.

Comunidade trabalha-se com calma. Não vamos assustar as pessoas que tem pouco tempo para colaborar.

@farribeiro sobre esse PR, se você citar uma fonte válida que diz porque padronizar isso é útil para o projeto, iremos dar o merge. Mas tem que ter um motivo com entrega de valor.

[farribeiro]

Coloque todos os arquivos em 777 e seja feliz!

[farribeiro]

Sabe o que causa a minha afirmação do 777 ? Brecha de segurança!

[MarceloCajueiro]

Tem algum com 777? Se tem, vale o PR para arrumar. O que você fez antes aqui, arrumou alguma brecha? Se arrumou, faltou explicar isso.

Tks.

[farribeiro]

Restore branch e re-abrindo PR para análise

[farribeiro]

Removido conflitos

[eberfreitas]

Oi pessoal. O que vamos fazer com este PR? Me parece que ficou faltando uma solução pro passo de darmos permissão 777 no folder de relatórios...

Alguma evolução neste aspecto @farribeiro? Vale lembrar que a instrução de usar 777 é pra desenvolvimento e talvez a gente possa deixar isso mais claro assim que tivermos os scripts de instalação (#232). Não sei se seria muito produtivo tentar resolver isso agora no Docker até pq é uma parte complicada de testar e validar.

Se nada mais for ser desenvolvido aqui, sugiro que o PR seja fechado pra gente partir pra próxima pendência.

[farribeiro]

Tenho fé que mesmo desenvolvendo precisa do princípio mais próximo de produção.

Bom, já que não deu em nada mesmo, autorizo fechamento