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.

Sign up for GitHub

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

Jump to bottom

V1 dev #4

Merged
merged 5 commits into from
Jun 6, 2024
Merged

V1 dev #4

merged 5 commits into from
Jun 6, 2024

Conversation

lpicci96
Copy link
Collaborator

@lpicci96 lpicci96 commented Jun 6, 2024

  • Adds function attribute last_version_fetched to function fetch_data in reader.py
  • Specifies numeric data types for each numeric column in parser.py
  • Add additional testing

@lpicci96 lpicci96 requested a review from jm-rivera June 6, 2024 10:55
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

jm-rivera
jm-rivera approved these changes Jun 6, 2024
View reviewed changes
Copy link
Contributor

@jm-rivera jm-rivera left a comment

Choose a reason for hiding this comment

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

nice!

@jm-rivera jm-rivera merged commit 689963c into main Jun 6, 2024
6 checks passed
@lpicci96 lpicci96 deleted the v1-dev branch June 6, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jm-rivera jm-rivera jm-rivera approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lpicci96 @codecov-commenter @jm-rivera