-
Couldn't load subscription status.
- Fork 121
Added 2021 and 2022 data with aanalysis and updated streamlit app #316
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
Conversation
Co-authored-by: Nikita Nikitababb036@gmail.com Co-authored-by: Leena leenagoyal2403@gmail.com
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome @NIKITA320495
some suggestions for you would be : adding description on the changes made in the PR.
incoperating bit more coding comments. so it will improve readbility and understanding for future developments.
🎉 Your pull request has been successfully merged! 🎉 Thank you for your valuable contribution to our project. Your efforts are greatly appreciated. Feel free to reach out if you have any more contributions or if there's anything else we can assist you with. Keep up the fantastic work! 🚀
Could you check and reply here regarding the Streamlit app and Streamlit on Snowflakes are same thing? @NIKITA320495
hello @sanjay-kv sir, please assign me the PR as well. As Nikita and I collaborated on this one as well. And yes, we'll make sure to add proper comments for readability from now on. As for Snowflakes, we never worked on it, so not sure.
leena just raise one more PR and link the PR to there. and I will assign same level. it has been noted leaderboard is not fetching two people on co-authored commits.
- Raise an new PR by changing readme file
- Explain in the description you worked on this PR with links
- I will merge and give the same level @NIKITA320495 has got.
Ok @sanjay-kv sir !
This is a sample PR for the issue recodehive#316 (recodehive#316) that I collaborated on. Due to some technical issues, my PR couldn't be merged, hence a sample merge is being created for fetching updated information on leaderboard.
No description provided.