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

Fix/version upgrades #95

Merged
merged 3 commits into from
Sep 29, 2023
Merged

Fix/version upgrades #95

merged 3 commits into from
Sep 29, 2023

Conversation

vminfant
Copy link
Collaborator

@vminfant vminfant commented Sep 29, 2023

Describe your changes

Upgrading pydantic, pandas versions to next major version and accommodating the changes related to the same.

Checklist

  • I have added/updated the necessary documentation for the changes.
  • I have added/updated the tests for the changes.
  • I have updated the poetry version. Refer version upgrades.
  • I have updated the version in __init__.py
  • I have updated the CHANGELOG.md

@vminfant vminfant requested a review from ckcogniter September 29, 2023 09:43
@codecov
Copy link

codecov bot commented Sep 29, 2023

Codecov Report

Merging #95 (d2ba808) into main (df930c2) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #95      +/-   ##
==========================================
- Coverage   96.19%   96.16%   -0.03%     
==========================================
  Files           4        4              
  Lines         315      313       -2     
==========================================
- Hits          303      301       -2     
  Misses         12       12              
Files Coverage Δ
cognite/cdffs/__init__.py 100.00% <100.00%> (ø)
cognite/cdffs/credentials.py 100.00% <100.00%> (ø)

@vminfant vminfant merged commit c337507 into main Sep 29, 2023
3 checks passed
@vminfant vminfant deleted the fix/version-upgrades branch September 29, 2023 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants