Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Update RadStation #162

Merged
merged 5 commits into from
Sep 24, 2024
Merged

Update RadStation #162

merged 5 commits into from
Sep 24, 2024

Conversation

Ichaie
Copy link

@Ichaie Ichaie commented Sep 22, 2024

Description

Atualiza o mapa RadStation, adicionando pequenas coisas faltando e aproveita para corrigir alguns nomes de grids de alguns mapas originais no Estação Pirata.


TODO

  • Testar a vaga para Mantis no rad

Changelog

🆑

  • Adiciona dresser para as salas dos membros de comando
  • Adiciona vaga para Mantis no RadStation
  • Corrige nomes para grids de alguns mapas

@rbertoche
Copy link
Owner

Deu um conflito, nao sei qual dos dois é pra manter

@Ichaie
Copy link
Author

Ichaie commented Sep 23, 2024

Provavelmente o conflito se deu por causa que mergiu o primeiro comit que corrige a localização dos mapas

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@rbertoche
Copy link
Owner

Foi isso mesmo Ichai, se vc for fazer mais de 1 PR precisa usar branches independentes pra nao causar isso, se um for derivado do outro da esse problema, um acaba causando conflito no outro mesmo sendo a mesma alteração
o github realmente deixa mt a desejar nessa operação né mas enfim nao é algo tao trivial quanto parece, quando rola um rebase, é "outro commit"

@rbertoche
Copy link
Owner

de novo...

@rbertoche
Copy link
Owner

ta vendo pq eu nao gosto de usar PR? as vezes atrapalha mt mais do que ajuda

@rbertoche
Copy link
Owner

eu mergi os outros e esse ficou com conflito de novo
ah vc ta fazendo tudo em um branch só. Não tem como
se vc tiver usando um branch, é 1 PR só
se quiser abrir 2 PR precisa de 2 branches sem um ser derivado do outro
mas por mim nao precisa abrir nem 1 PR nem 2

@rbertoche
Copy link
Owner

eu nao gosto de fazer squash mas o git me obriga

@rbertoche rbertoche merged commit fce9789 into rbertoche:pirata Sep 24, 2024
3 of 6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants