search

kodsnack / advent_of_code_2017

Contribute your solutions to Advent of Code 2017 and be inspired by others.
51 stars 98 forks source link

Var tog min kod vägen? #233

Open Masssssy opened 6 years ago

Masssssy commented 6 years ago

Min submodul fanns med igår och i morse men nu är den puts väck :( Vad har hänt där tro?

bjoreman commented 6 years ago

Antar att någon lyckades pusha en senare uppdaterad version av submodules som inte innehöll din modul och dessutom inte sågs som en konflikt? Submodules känns lite osäkra i ett repo med såpass många bidragsgivare.

olssonm commented 6 years ago

Verkar ha skett ett märkligt misstag där; ser ut som @mrinlumino råkade radera din submodul i commit https://github.com/kodsnack/advent_of_code_2017/pull/218/commits/afab6642244ec890e286dcca42592bd2f38243f2 (Submodule masssssy-python deleted from 83b2e4) och ändringen merge:ades sedan in i PR #218.

Nog ganska lätt hänt tyvärr =/ Men nu är du tillbaka i alla fall =)

bjoreman commented 6 years ago

Ja, det är ju skönt att det går hyfsat lätt att reparera skadan. Men störigt ändå.

bjoreman commented 6 years ago

Nästan värt att behålla denna öppen som en liten påminnelse. Chansen är ju god att samma sak händer någon mer gång.

mrinlumino commented 6 years ago

Oj, ber så mycket om ursäkt! Förstår inte vad som hände ärligt talat. Jag hade bara lagt till två filer i min egen commit och sedan gjort en fetch & merge från kodsnack-repot innan jag committade. Måste hänt något märkligt i min egen merge. Jag borde ju sett vad som var på gång när jag lade pull-requesten men var nog lite för snabb. Lovar att hålla ögonen öppna innan jag pushar något idag.

Masssssy commented 6 years ago

Min tanke var ju att det skulle bli smidigare med en submodul men det verkar vara mer meck i praktiken trots all den fina teorin :D

bjoreman commented 6 years ago

Samtidigt tycker jag det funkar rätt bra med tanke på hur många som använder submoduler.

mrinlumino commented 6 years ago

Gjorde en clean checkout nu och tänker strunta i att försöka synka ner alla andras ändringar till min fork så skall jag förhoppningsvis inte orsaka mer elände framöver!

bjoreman commented 6 years ago

Äsch, vi lär oss ju alla saker på kuppen. Kör på!

k0nserv commented 6 years ago

Så här gör jag när jag uppdatera

git checkout master
git fetch --all
git reset --hard upstream/master
cd k0nserv-rust
git pull
cd ..
git add k0nserv-rust
git commit
git push origin master