fredrikbixo / stockpicker Goto Github PK

Jag tycker projektbeskrivningen är bra, den är kort och koncis och man förstår vad projektets syfte är. Dock skulle den kunna kompletteras med följande (från Rics lydelse):
The Project Description should contain:

• Documentation on how to install and use the project (this will evolve)
• Give a description of your testing strategy

Projektplanen (milestones) för projektet har säkert varit användbar för de som deltar i projektet. Men som utomstående skulle jag önska lite mer beskrivande text om varje delmål. Bifogar även här Rics lydelse angående ämnet.

The Project Plan should be expressed using Features/Milestones that are:

• Clear units of achievable work described as Features in the Issue Tracker
• Sensible groups of features/bugs into Milestones (by time, e.g. "Week 1", or by function, e.g. "Backend", or a schema you prefer)
• Discussed and updated and closed as the project evolves

Har använts och jag ser inga problem med hur det använts.

Rated whole SP500 index

Tror inte jag har något att anmärka på här. Enkla kommentarer och de commits jag kollat på ser rimliga ut (inte för mycket kod som uppdaterats mellan commits osv.)

Vi måste fundera på hur vi på bästa vis kan gå igenom en lista med ex alla SP500-bolag. Löses detta enklast med en for-loop och alla bolagen i en array?

Issues verkar i det här projektet mer kopplade till milestones än till att lösa buggar. Vad jag ser är det endast en issue som saknar koppling till en milestone (Gå igenom alla aktier #6). Svårt att säga om det har behövts användas mer än så. "Gå igenom alla aktier #6" tycker jag är ett bra exempel på hur funktionen kan användas. Om något så kanske funktionen borde använts mer.

Milestones och issue tracker har använts och det är bra. Det ända jag har att anmärka på här, är som jag redan nämnt i Level 1 att det saknas förklarande text om varje milestone eller Issue.