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

Core-file #22

Merged
merged 16 commits into from
May 5, 2022
Merged

Core-file #22

merged 16 commits into from
May 5, 2022

Conversation

eccentricOrange
Copy link
Owner

Some of the changes will spill over to the CLI.

Notable deviations from issue:

  • used yield a lot more
  • broke up udl string parsing
  • made custom exceptions file
  • added support for logging costs
  • data retreival is more flexible

@eccentricOrange eccentricOrange linked an issue May 5, 2022 that may be closed by this pull request
Incorporate CRSE and Reddit feedback #20
Closed
@eccentricOrange eccentricOrange merged commit 2526462 into 20-incorporate-crse-and-reddit-feedback May 5, 2022
@eccentricOrange eccentricOrange deleted the core-file branch May 5, 2022 20:39
@eccentricOrange eccentricOrange added enhancement New feature or request CRSE Feedback from the Code Review Stack Exchange labels Jun 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
CRSE Feedback from the Code Review Stack Exchange enhancement New feature or request
Projects
None yet
Milestone
First round of feedback from CRSE
Development

Successfully merging this pull request may close these issues.

Incorporate CRSE and Reddit feedback
1 participant
@eccentricOrange