Skip to content
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

Definiera origin-url för streaming-server #76

Open
oleander opened this issue Mar 6, 2012 · 4 comments
Open

Definiera origin-url för streaming-server #76

oleander opened this issue Mar 6, 2012 · 4 comments

Comments

@oleander
Copy link
Member

oleander commented Mar 6, 2012

Relaterat till #75

Hur bör git-urlerna se ut för en given lab?

Exempel

git remote add water http://water.chalmers.se/lab_groups/12/submissions

@Tarrasch
Copy link
Contributor

Tarrasch commented May 2, 2012

Relaterad till #152. (varför har denna low priority, borde vara high i nuläget)

@oleander, jag föreslår att vi även har med lab_groups i urlen och inte bara given_course och labs. Varför? För

  • Vi kan unikt identifiera en repo, även en handledare kan ju använda samma url då.
  • Det är simplare att göra queriesarna för vår grack-plugin. Att identifera användaren och indentifiera repon har nu separerats (förstår du vad jag menar?).
  • Vi behöver inte förlita oss på invarianten
  • git kommer ge väldigt konstiga fel ifall du jobbar på en labb, sen plötsligt har någon annan bytt ut din lhg till en annan i någon annan labbgrupp, då kommer git plötsligt klaga på ingen common ancestor commit, förvirrande för användaren.

Jag vet att du hade starka åsikter om att vi inte ska ha lab_groups med iom det bara finns 1 lhg ty invarianten, fast vi måste ju inte utnyttja invarianten. Vad tycker du i dagsläget?


En helt annan sak, varför har vi labs i plural och inte lab? Är något mer en konvention? Det är ju mer naturligt med singular i detta fall eller?

@Tarrasch
Copy link
Contributor

Tarrasch commented May 4, 2012

Ok, @oleander, jag märker att du inte svarat än och jag misstänker att Jesper vill att jag implementerar lhg#clone_url asap. Så jag beslutar att köra på det jag sa på egen hand, så såvida du inte har en väldigt bra anledning på att vi inte gör som jag sa så kan du klaga.

Förresten, jag fortsätter med plural (sådana är routesen i rails-appar, eller hur?), vad tycker du? Fast egentligen tycker jag singular är mycket snyggare. Du säger ju "kurs fem", inte "kurser fem".

Jag väntar på att du Linus godkänner det jag sagt och sedan stänger issuen.

@oleander
Copy link
Member Author

oleander commented May 4, 2012

Hehe, jag fick 28 PM här på Github under gårdagen. Har inte hunnit gå igenom alla än.

Du kan ta och lägga till lab_groups, de blir bra.
Singularis fungerar med.

@Tarrasch
Copy link
Contributor

Tarrasch commented May 5, 2012

Ok, vi har bestämt argumenten till urlen, nu funderar vi bara på sing- vs plural. Så nu är denna bara low priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants