Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

removido comentarios para diminuir tamanho do projeto em nugets #48

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

danilobreda
Copy link
Collaborator

removido comentarios para diminuir tamanho do projeto em nugets

@danilobreda danilobreda merged commit 53762f0 into master Oct 29, 2024
2 checks passed
@adeniltonbs
Copy link
Contributor

Bom dia @danilobreda, saberia informar se foi verificado possíveis violações da licença da biblioteca ao enviar esse PR?
Obs.: esse tipo de comentário é ignorado pelo compilador: https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/xmldoc/

@danilobreda danilobreda deleted the remocaocomentariosv2 branch December 18, 2024 15:37
@danilobreda
Copy link
Collaborator Author

danilobreda commented Dec 18, 2024

@adeniltonbs tudo certo? O motivo foi meios técnicos o que a o LGPL 2.1 permite. O objetivo era testar e verificar, tivemos melhoras de tamanho e de código fonte também. Quanto violação de licença, o Hercules hoje tem sua vida própria e não somos mais um fork dependente do antigo Zeus, aprovado pela gestão do github viramos um projeto separado, com suas próprias contribuições e melhorias.
Agradeço o caminho até aqui no desenvolvimento e contribuição. A referencia aos desenvolvedores originais estão no histórico, que não foi perdido e em README com o texto do fork realizado na época (segue imagem).
image

image

Infelizmente com essa modificação acho que complicamos como o antigo Zeus realizava pequenas copias de modificação do código do Hercules. Isso é uma pena. :/

@marcosgerene
Copy link
Collaborator

Aquele tipo de texto dentro do arquivo .cs precede a existência do git.

A remoção não apaga o histórico de quem contribuiu, so reduz o volume de texto desnecessário reduzindo o
tamanho do projeto em si. Melhoria tecnica clara.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants