Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

students_classes.csv #17

Merged
merged 7 commits into from
Oct 25, 2023
Merged

students_classes.csv #17

merged 7 commits into from
Oct 25, 2023

Conversation

jvdcf
Copy link
Owner

@jvdcf jvdcf commented Oct 23, 2023

Warning

Still in draft
Edit: É preciso pagar 4$ por mês para marcar um pull request como draft, lol

O código já está implementado, mas ainda falta testar e documentar.

@jvdcf jvdcf self-assigned this Oct 23, 2023
src/studentsClasses.hpp Outdated Show resolved Hide resolved
src/studentsClasses.cpp Show resolved Hide resolved
src/studentsClasses.cpp Outdated Show resolved Hide resolved
src/studentsClasses.cpp Outdated Show resolved Hide resolved
@jvdcf jvdcf requested a review from DuarteSAssuncao October 24, 2023 06:07
@jvdcf jvdcf added Bug Erros de runtime, etc. Foco O foco principal do projeto por agora é acabar esta tarefa and removed Bug Erros de runtime, etc. labels Oct 24, 2023
Copy link
Collaborator

@guilherme-ds-matos guilherme-ds-matos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acho que é só preciso mesmo usar o stable sort. Para além disso, se todas as classes tiverem UcCode e ClassCode, acho que AppCPU devia ser uma superclasse de todos os outros para não repetirmos código.

@guilherme-ds-matos guilherme-ds-matos merged commit 7e11785 into main Oct 25, 2023
1 check passed
@jvdcf jvdcf deleted the parsec_SC branch October 25, 2023 15:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Foco O foco principal do projeto por agora é acabar esta tarefa
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants