beanie icon indicating copy to clipboard operation
beanie copied to clipboard

Index Error Fix

Open gaurav25122000 opened this issue 1 year ago • 1 comments

Added conversion for float to int for existing indexes while being fetched from mongo database due to pymongo check of indexes being 1,-1 or 2d

gaurav25122000 avatar Dec 19 '23 21:12 gaurav25122000

Hi @gaurav25122000 Thank you for the PR! Could you please add a test for this?

roman-right avatar Dec 19 '23 23:12 roman-right

To help other potential contributors to understand the code and the change, it would be nice to give a context in the pull request description or a reference to the issue where the context is given.

About the commit message:

Added conversion for float to int for existing indexes while being fe…
…tched from mongo database due to pymongo check of indexes being 1,-1 or 2d

According to git conventions, the subject should be in imperative present tense. The current subject is not a sentence and ambiguous (added is not a past tense verb in correct grammar).

Convert float indexes to int while being fetched from mongo database due to pymongo check of indexes being 1,-1 or 2d

This is already in imperative present tense, but for me, it is not clear what the part due to... means and it is too long. The subject line should give a hint why the change was needed and the details part of the commit message should explain why in simple technical English. What the change is that is visible in the diff.

These could cost some effort for the contributor, but would save time for others and speed up the development of the project.

matez0 avatar Jan 25 '24 10:01 matez0

Hi @gaurav25122000 Thank you for the PR! Could you please add a test for this?

Can't find the test cases for this particular function

gaurav25122000 avatar Jan 27 '24 14:01 gaurav25122000

Can't find the test cases for this particular function

This is true. Could you please add one? @gaurav25122000

roman-right avatar Feb 08 '24 20:02 roman-right

This PR is stale because it has been open 45 days with no activity.

github-actions[bot] avatar Mar 25 '24 01:03 github-actions[bot]

This PR was closed because it has been stalled for 14 days with no activity.

github-actions[bot] avatar Apr 08 '24 01:04 github-actions[bot]