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

Proposal to refactor Reader and Type classes #108

Merged

Conversation

Irineu333
Copy link
Member

@Irineu333 Irineu333 commented Nov 17, 2023

Description

A proposal to improve the Type class. It also extracts state and type reading logic for extensions.

I didn't send it directly in a commit to #100, because it's a proposal. We may not go through with it.

Improvements

  • Refactor Type enum class to selead class
  • Simplify Reader class
  • Improve and extract node read logic for extensions

Fixes

@Irineu333 Irineu333 added the refactor Improvement without changing behavior label Nov 17, 2023
@Irineu333 Irineu333 requested a review from PatrykMis November 17, 2023 04:54
@Irineu333 Irineu333 self-assigned this Nov 17, 2023
@Irineu333 Irineu333 modified the milestone: MVP Nov 20, 2023
@Irineu333 Irineu333 changed the title Proposal to refactor type classes Proposal to refactor Reader and Type classes Nov 20, 2023
@Irineu333 Irineu333 added the subtask Subtask of an issue. label Nov 20, 2023
@Irineu333 Irineu333 force-pushed the refactor/improve-type-class branch 4 times, most recently from 0c33362 to 0a0f8f3 Compare November 20, 2023 14:24
@Irineu333 Irineu333 modified the milestone: MVP Nov 20, 2023
@Irineu333 Irineu333 force-pushed the refactor/improve-type-class branch 3 times, most recently from 80a6bc7 to aa0e3d5 Compare November 21, 2023 02:37
@Irineu333 Irineu333 force-pushed the refactor/improve-type-class branch 5 times, most recently from 3d9c9eb to 30e2263 Compare November 29, 2023 05:57
@PatrykMis PatrykMis force-pushed the feature/ISSUE#97-improve-reader-class branch from 9c9774b to 3bb9d58 Compare November 29, 2023 22:44
@PatrykMis PatrykMis force-pushed the refactor/improve-type-class branch from 45f8a63 to adcbfba Compare November 29, 2023 23:01
@PatrykMis PatrykMis force-pushed the feature/ISSUE#97-improve-reader-class branch from 3bb9d58 to 956649b Compare November 30, 2023 03:22
@Irineu333 Irineu333 force-pushed the refactor/improve-type-class branch 2 times, most recently from 8eb3d80 to ae6d8fd Compare November 30, 2023 03:31
@PatrykMis PatrykMis force-pushed the feature/ISSUE#97-improve-reader-class branch from 956649b to 3bb9d58 Compare November 30, 2023 03:38
@Irineu333 Irineu333 force-pushed the refactor/improve-type-class branch from ae6d8fd to 41970d9 Compare November 30, 2023 03:39
@Irineu333 Irineu333 force-pushed the refactor/improve-type-class branch from 41970d9 to c09dccb Compare November 30, 2023 03:57
@Irineu333 Irineu333 merged commit 840f63f into feature/ISSUE#97-improve-reader-class Nov 30, 2023
@Irineu333 Irineu333 added feature New feature or request refactor Improvement without changing behavior and removed refactor Improvement without changing behavior feature New feature or request labels Nov 30, 2023
@Irineu333 Irineu333 mentioned this pull request Nov 30, 2023
@Irineu333 Irineu333 deleted the refactor/improve-type-class branch January 3, 2024 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Improvement without changing behavior subtask Subtask of an issue.
Projects
Status: Ready For Production
Development

Successfully merging this pull request may close these issues.

2 participants