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

Typo fix in box_list #11032

Closed
wants to merge 1 commit into from
Closed

Typo fix in box_list #11032

wants to merge 1 commit into from

Conversation

laxmareddyp
Copy link
Collaborator

Type of change

  • Documentation update

Checklist

@laxmareddyp laxmareddyp requested review from fyangf and luotigerlsx and removed request for saberkun, yeqingli, arashwan and rachellj218 June 27, 2023 22:39
@laxmareddyp laxmareddyp added the models:official models that come under official repository label Jun 27, 2023
@luotigerlsx
Copy link
Member

Why we want to make these changes ? To me, it's not typo and more like preference choices consistent across the file. Are we trying to make alignment across the codebase ?

@laxmareddyp
Copy link
Collaborator Author

Why we want to make these changes ? To me, it's not typo and more like preference choices consistent across the file. Are we trying to make alignment across the codebase ?

I believe it would be good to maintain consistency. However, if it doesn't make sense to proceed with this pull request, we can kindly consider closing it.

@laxmareddyp laxmareddyp deleted the laxmareddyp-patch5 branch June 30, 2023 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
models:official models that come under official repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants