-
Notifications
You must be signed in to change notification settings - Fork 13
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
Removed Comments an unused imports #15
Conversation
@A91y issue no 2, about removing unused code. |
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.
Do not update sitemap file and don't upload sitemap-0.xml
@A91y I made the necessary changes, can u merge the PR. |
@Zaid-B24 don't modify .gitignore and sitemap.xml |
Made sitemap to original file, also changed gitignore to original file
@A91y made the changes. |
@Zaid-B24 use of such non-descriptive commit message is not allowed. And when you make a PR that fixes some issue, you need to link the issue to the PR. |
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.
lgtm
Thanks @Zaid-B24 |
thank you @A91y |
Made the necessary changes by removing unused code, also removed commented codes and unused imports