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

Adiciona Java na listagem de Linguagens, um artigo sobre Java FX, um sobre JavaEE e outro sobre Vraptor 4 #44

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pedro-hos
Copy link

Adiciona alguns artigos desenvolvimento Java para contribuir com a documentação já existente

@thulioph
Copy link
Member

Obrigado pela contribuição @pedro-hos, mas você poderia modificar a sua mensagem de commit para ficar de acordo com o nosso padrão? Após a correção o merge será feito 😄

@cerebrobr/editores 👍

@fdaciuk
Copy link
Contributor

fdaciuk commented May 10, 2015

Perfeito! Só ajustar o título do commit, como o @thulioph falou :D
Fora isso, 👍

@pedro-hos pedro-hos changed the title Adicionando Artigos Java Adiciona Artigos Java na documentação May 11, 2015
@pedro-hos
Copy link
Author

@cerebrobr/editores Essa edição que vocês precisavam? Se não estiver de acordo, realizo as alterações. Valeu 👍

@fdaciuk
Copy link
Contributor

fdaciuk commented May 11, 2015

@pedro-hos é nas mensagens de commit mesmo: https://github.com/cerebrobr/artigos/pull/44/commits

:)

@nicholasess
Copy link
Contributor

@pedro-hos ajusta a mensagem de commit

@pedro-hos pedro-hos changed the title Adiciona Artigos Java na documentação Adiciona Java na listagem de Linguagens, um artigo sobre Java FX, um sobre JavaEE e outro sobre Vraptor 4 Sep 9, 2015
@pedro-hos
Copy link
Author

@nicholasess está de acordo agora? rs

@fdaciuk
Copy link
Contributor

fdaciuk commented Sep 9, 2015

@pedro-hos, são essas mensagens que tem que mudar: https://github.com/cerebrobr/artigos/pull/44/commits

Para fazer isso, aqui tem as instruções: https://help.github.com/articles/changing-a-commit-message/

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.

4 participants