Skip to content

Conversation

@vitor-scheffer
Copy link
Contributor

Descrição e Solução

  • Integrando a biblioteca SnapshotTesting através do Swift Package Manager

Checklist:

  • Não adiciona código duplicado
  • Não contém código comentado
  • Não contém código WIP
  • Teste Unitário Implementado

Evidências:

iPhone SE iPhone 13 Pro Max
print print

Screenshot 2022-12-06 at 22 51 46

Integrando biblioteca SnapshotTesting através do SwiftPackageManager.
Ajustando class DeliveryAppTests.
@FernandoVGMonteiro
Copy link
Contributor

Quando o PR da Julia foi mergeado, acabou gerando um conflito no seu PR dentro do arquivo de projeto (project.pbxproj).

Esse é um dos conflitos mais comuns que aparece. O arquivo de projeto configura tanto dependências do projeto (que é o caso da alteração do seu PR, onde uma dependência do SPM foi adicionada), quanto a hierarquia de arquivos dentro do XCode (que é o caso do PR da Ju, que adicionou dois novos arquivos no projeto). Como o conflito é com linhas novas que foram adicionadas nos dois PRs, na hora de resolver o conflito é só manter as duas alterações! :)

Tem um artigo sobre a resolução de conflitos no Notion da DevPass: https://www.notion.so/devpass/Conflitos-no-Git-70d889a06f264c7b888769c380632bf6

Ou você pode usar a própria interface aqui no git pra resolver o conflito, dá uma olhada nessa documentação aqui: https://docs.github.com/pt/pull-requests/collaborating-with-pull-requests/addressing-merge-conflicts/resolving-a-merge-conflict-on-github

@vitor-scheffer
Copy link
Contributor Author

Beleza, entendi.

Fiz um novo commit resolvendo os conflitos acredito que agora está tudo certo, eu preciso gerar um novo PR?

@FernandoVGMonteiro
Copy link
Contributor

Beleza, entendi.

Fiz um novo commit resolvendo os conflitos acredito que agora está tudo certo, eu preciso gerar um novo PR?

Não precisa não!! Já apareceu sua alteração aqui no PR! :)

Copy link
Contributor

@FernandoVGMonteiro FernandoVGMonteiro left a comment

Choose a reason for hiding this comment

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

Boa mano!!!

Aprovado! Vou fazer o merge :)

@FernandoVGMonteiro FernandoVGMonteiro merged commit d7d1614 into devpass-tech:main Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants