-
Notifications
You must be signed in to change notification settings - Fork 1
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
Preparação para a release de uma 1.0.0 #10
Conversation
<packaging>jar</packaging> | ||
<groupId>br.ufsc.bridge</groupId> | ||
<artifactId>metafy</artifactId> | ||
<version>0.2.1-SNAPSHOT</version> |
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.
vai fazer o release dps desse pr?
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.
uhum
.github/workflows/maven.yml
Outdated
@@ -3,28 +3,25 @@ name: Java CI | |||
on: | |||
push: | |||
branches: | |||
- master | |||
- main | |||
- main-v1 |
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.
aqui precisa ficar o branch main? ai vai rodar a action no branch main com java 17?
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.
vou tirar essa parte do on push
coloquei de viagem pq tinha no bold, mas como aqui vai ficar similar ao que temos no PEC (PRs e dois branchs principais), vou deixar estilo pec
Recomendo revisar com a opção "hide whitespace" selecionada (https://github.com/laboratoriobridge/metafy/pull/10/files?w=1), porque o formatter do bridge fez várias alterações de espaço nos arquivos.
De uma parte mais geral do repositório, fiz o seguinte:
main-v1
master
paramain
main
emain-v1
Com dois branchs, a ideia é a seguinte:
main
main-v1
Nesse PR em específico:
Removi uso do plugin do javadoc, porque não tinha documentação escrita e isso só poluía o log de build com warningPontos de atenção para o futuro:
main
, será preciso atualizar a action do github de forma similar ao 0171a9c