Skip to content

Sets #21

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

Merged
merged 2 commits into from
Nov 13, 2021
Merged

Sets #21

merged 2 commits into from
Nov 13, 2021

Conversation

pgjbz
Copy link
Contributor

@pgjbz pgjbz commented Nov 12, 2021

Dando sequencia na explicação das coleções do Java, agora venho com o Set e a explicação de 3 implementações diferentes da interface

Copy link
Collaborator

@FerroEduardo FerroEduardo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conteúdo muito bacana, só preciso de pequenas correções.

  • Observei que os links da documentação são de versões diferentes. Para deixar padronizado, peço que deixe todos eles na versão 11 do JDK.

@FerroEduardo
Copy link
Collaborator

Talvez algo que nós podemos fazer futuramente é organizar os links de documentações e padronizar para a versão LTS mais recente. Creio que isso ajude a evitar falhas ao comentar sobre métodos.
Digo isso, pois existem métodos comentados nesse PR que não estão presentes no JDK 8, como o Set.of(). Não verifiquei se isso ocorreu no PR anterior, mas é algo que pode ser evitado.

Não veja essa mensagem como uma crítica, é mais um comentário sobre o futuro do repositório. Acho até que poderia ser uma issue válida.

@pgjbz
Copy link
Contributor Author

pgjbz commented Nov 12, 2021

Talvez algo que nós podemos fazer futuramente é organizar os links de documentações e padronizar para a versão LTS mais recente. Creio que isso ajude a evitar falhas ao comentar sobre métodos. Digo isso, pois existem métodos comentados nesse PR que não estão presentes no JDK 8, como o Set.of(). Não verifiquei se isso ocorreu no PR anterior, mas é algo que pode ser evitado.

Não veja essa mensagem como uma crítica, é mais um comentário sobre o futuro do repositório. Acho até que poderia ser uma issue válida.

Concordo em padronizar para a ultima versão LTS, porém temos que tomar cuidado, pois a 8, 11 e 17 são LTS e muitas vezes o Java 8 ainda é usado na maior parte dos projetos, o que eu fiz agora foi especificar que o Set.of só existe em versões acima do Java 8, creio q inicialmente vou tomar mais este cuidado nas proximas colaborações em relação a isso.

@pgjbz pgjbz requested a review from FerroEduardo November 12, 2021 11:17
@FerroEduardo FerroEduardo merged commit 5f5113c into paulorievrs:master Nov 13, 2021
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.

2 participants