-
Notifications
You must be signed in to change notification settings - Fork 1
/
Copy pathcode_review_20130516.txt
52 lines (33 loc) · 1.43 KB
/
code_review_20130516.txt
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
### Palvelin ###
- universal-supportti
- copy-paste -helvetti
- halutaan esim. "tähän kurssiin git submodulena tämän kielen tuki tuosta reposta"
- course refreshin skriptattavien osien sandboxaus?
- siirretään toiseen branchiin?
- rename src-min to 'ace' or something reasonable
- pastenäkymä:
- tabit näyttämään enemmän tabeilta
- tabeille klikkauskäsikursori
- sama testeille
- kommenttikenttä ja send-nappi nätimmin
- kommenttikenttä isommaksi
- harkitse urlin muutosta
http://tmc.jamo.fi/exercises/142.json
=>
http://tmc.jamo.fi/exercises/142/submissions.json
- acts_as_api yhdenmukaisesti käyttöön kaikkialla
- rake [re]compile ja clean ctestrunnerille
- selvitä lib/submission_packager/makefile_c.rb:n viittaus junit runneriin ja
junit runnerin tmcee-sotkut
- rename test_case_runs.backtrace -> test_case_runs.detailed_message tms
- pastessa virheen näyttäminen paremmaksi
- ehkä ei modaali
- ei ainakaan fixed size modaali
- siirrä TMCeeTestRunner testmycode-organisaatioon ja nimeä se uudelleen c-test-runner tms
- siirrä tmc-c-course-template testmycode-organisaatioon
- siisti ja integroi päärepoon https://github.com/TMCee/Documentation/blob/master/asennus.md
### NB-plugin ###
- nimentä: backtrace -> detailed_message tms NB:n tulosnäkymässä
- siivoa RunTestsLocallyAction
- request review ja pastebin -dialogien sijainti
- pastebin-urlille copy to clipboard -nappi