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

Possible improvement for method 'sorted_nicely' #2

Open
OshinoMoumou opened this issue May 30, 2021 · 2 comments
Open

Possible improvement for method 'sorted_nicely' #2

OshinoMoumou opened this issue May 30, 2021 · 2 comments

Comments

@OshinoMoumou
Copy link

Greetings!
I noticed that method sorted_nicely referred a StackOverflow post, while a recent comment to it noticed that it doesn't work for sorting sets of version numbers alphanumerically; e.g., v1.0.1, v3.5.3, v3.2.4.
I'm trying to learn the usefulness of such small updates on StackOverflow. Would this comment help improve your code? I understand that might not happen in real life situation. In that case, do you think this comment can help prevent future defect (if the code were reused somwhere else)?
I'll really appreciate it if you could kindly give me some feedback or suggestions. Thank you very much for your time.
Have a nice day!

@justincc
Copy link
Collaborator

justincc commented Jun 1, 2021

Hi @OshinoMoumou. Please could you be more specific as to what you're referring to in this codebase.

@OshinoMoumou
Copy link
Author

Hi @justincc ! Sorry for the late reply. For method 'sorted_nicely' I mean the one in analysis/opensimulator-script-analyzer/src/ossa.py. I think it's supposed to sort real-life versions including those start with 'v', e.g., v1.0.1, v3.5.3, v3.2.4. However the comment I mentioned in the issue noticed that the method could fail in such case.
Though that might not be a problem in your case, I'm studying on the usefulness of StackOverflow comments and noticed your reference. What I really want know is that whether you think this comment can help imporve the code. I'd be glad to know your opinion.
Have a nice day :)

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

No branches or pull requests

2 participants