Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Anders Nygren
    @litemerafrukt
    Ändrat namn och överfört till CAnax organisation. Såg då att alla moduler har liten bokstav. Tänkte inte alls på det. Jag verkar inte ha rätt att ändra just den saken. Ser ingen settings-tab på CAnax-projekt.
    Mikael Roos
    @mosbth
    Jag kunde göra settings - rename. Namnet mappar mot packagist-modulen anax/uri.
    Anders Nygren
    @litemerafrukt
    Fint! Så uri finns på packagist nu? Då sätter jag igång med url och sedan att titta på request.
    Mikael Roos
    @mosbth
    Nu finns det där och jag la till badges i README.
    Mikael Roos
    @mosbth
    Om du gör url mot uri, gör det inledningsvis i en ny separat klass, så kan vi ha den gamla och nya kodbasen på samma plats och sen diskutera vilket som blir bäst.
    Vi behöver vara försiktiga när vi ändrar i befintlig kodbas så vi är bakåtkompatibla och inte inför nya buggar.
    Mikael Roos
    @mosbth
    Vet du, vi behöver prata lite om hur vi bäst utvecklar tillsammans, hur vi löser git o sånt. Låt oss ta ett samtal innan du sätter igång med Url, vi behöver en halv plan.
    Anders Nygren
    @litemerafrukt
    Absolut. Min dator har dock just gett upp en aning. Ska testa klassikern omformatering. Så jag börjar inte med Url de närmaste dagarna.
    Mikael Roos
    @mosbth
    Vi kanske kan ha en hangout nån gång nästa vecka?
    Anders Nygren
    @litemerafrukt
    Tisdag funkar bra för mig.
    Mikael Roos
    @mosbth
    Funkar efter luncheller kanske vid 15?
    eller 14 eller nåt...
    Anders Nygren
    @litemerafrukt
    14 funkar utmärkt
    Mikael Roos
    @mosbth
    bokat
    Anders Nygren
    @litemerafrukt
    :+1:
    Anders Nygren
    @litemerafrukt
    Halloj! HO-9? Hinner jag göra kaffe först?
    Mikael Roos
    @mosbth
    kaffe blir bra, hangout öhhh 7?
    Anders Nygren
    @litemerafrukt
    Oki, kommer om 3 minuter
    Anders Nygren
    @litemerafrukt
    Tjipp! Jag tyckte det lät som en bra ide att ha en enda funktion för startsWith. Så jag tänkte ta lite funderingar kring detta...
    För det första, jag skulle villja att metoden ska heta just startsWithoavsett om det är ett eller flera argument.
    Det är en ‘breaking’ change om jag tar bort startsWithAnymen paketet används ju inte ännu. Ska jag ändå sätta @depricated som vi diskuterade eller skippa det denna gången?
    (kanske ska testa bara för att :smile: )
    Vidare skulle jag vilja använda
    public function startsWith(...$strArr)
    Anders Nygren
    @litemerafrukt
    Du vet hur den fungerar :smile: Då gör jag ingen check i metoden om det är en array eller inte, det blir ju en array oavsett. Finns det någon uppenbar nackdel med att göra på det sättet (som jag inte tänkt på)? Personligen tycker jag det blir ett snyggt användargränssnitt.
    Sist men inte minst… Om jag skulle ändra till en foreachistället för array_reduce? Vill ju inte förvirra mina läsare i onödan :smile:
    Men äh. Jag tar bara och gör en ny branch och skriver lite så ser vi hur det ser ut sen :smile:
    Anders Nygren
    @litemerafrukt
    Såg i och för sig nu att paketet inte är taggat med 1.0.0, så breaking change måste vara ok.
    Mikael Roos
    @mosbth
    Breaking change innan release är helt okey, dessutom är modulen halvintern så länge vi integrerar den mot Url och Request.
    Anders Nygren
    @litemerafrukt
    :+1:
    Mikael Roos
    @mosbth
    Klurig idé med splash operator, jag tror jag kan använda det trixet på fler platser för att undvika mixed incoming parameters, det kanske inte är en så dum idé.
    Anders Nygren
    @litemerafrukt
    Lite klurigheter får man använda. Som elvis operator och spread operator ;)
    Mikael Roos
    @mosbth
    Jag gillar idé med en metod som är lite flexibel, så som startsWithblir. Det blir lite mer kod i just den funktionen men tittar man på den som använder metoden (och den kan ju användas på många platser) så känns det som man vinner i läsbarhet och enkelhet i användandet) . Så jag tycker det är en god idé att testa konceptet.
    Mikael Roos
    @mosbth
    gott, disabla lokalt bra grej
    ups, skulle till ircen
    Anders Nygren
    @litemerafrukt
    hehe
    Anders Nygren
    @litemerafrukt
    Tjipp. Händer inte mycket med denna just nu. Det har kommit en njursten och infekterad njure emellan. Senaste veckan har varit mycket morfin.
    Planen är fortfarande att uri v1 och url med uri ska vara på plats innan midsommar.
    Mikael Roos
    @mosbth
    Då får vi se om även jag kan komma loss och göra lite roliga saker tills dess, sitter fast i "shaving the yak"-liknande, som vanligt.
    Anders Nygren
    @litemerafrukt
    jag var tvungen att googla det där :smile:
    Mikael Roos
    @mosbth
    :) även jag fick googla det en gång i tiden, men passande uttryck till mycket av det man gör
    Anders Nygren
    @litemerafrukt
    Min macbook gav upp mitt i alltihopa så jag ligger efter. Tror dock fortfarande på en pull-request på Url-modulen innan midsommar :wink:
    Vi diskuterade arbetssätt för moduler. Det jag gjorde på uri var att jag skapade en ny gren på github och klonade hela repot. Sedan jobbade jag på den nya grenen och comittade nu när det känns ganska klart.
    Nu gjorde jag en pull request på github från den grenen till master. Då kunde jag även sätta en "reviewer". Testade sätta dig som reviewer trots att jag kanske skulle kunnat godkänna det själv.
    Kanske är det ett fungerande arbetssätt? Kändes smidigt i alla fall.
    Anders Nygren
    @litemerafrukt
    Om du godkänner pull-requesten på Uri taggar jag den med 1.0.0
    Under tiden sätter jag igång med Url-modulen.
    Mikael Roos
    @mosbth
    Kikade kort på den, godkände, såg codereview-grejen också. Låt se hur det funkar och ser ut när Url-modulen ändras.