Skip to content

KeyError Fix, Bug Report, #138#139

Merged
cbeddow merged 10 commits intomainfrom
Rubix982/KeyError-Fix
Mar 30, 2022
Merged

KeyError Fix, Bug Report, #138#139
cbeddow merged 10 commits intomainfrom
Rubix982/KeyError-Fix

Conversation

@Rubix982
Copy link
Contributor

Fix introduced for KeyError, as per #138.

This PR introduces,

  1. dateutil installation to mimic code provided
  2. Introduced dump folder and data for tests
  3. Wrote test for interface
  4. Makefile policy introduced

B
Installed dateutil, updated pytest.ini/Makefile for deprecation warnings

Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 30, 2022
Signed-off-by: Saif Ul Islam <saifulislam84210@gmail.com>
@cbeddow cbeddow merged commit bffba6e into main Mar 30, 2022
@cbeddow cbeddow deleted the Rubix982/KeyError-Fix branch March 30, 2022 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants