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

Added SQL and NoSQL database and removed table #244

Merged
merged 3 commits into from
May 16, 2024

Conversation

SaileePhal
Copy link
Contributor

IMPORTANT: When creating your Pull Request, please use the following format for the title: Category - Resource Name & Section

PR Title samples:

  • DBMS (Database) - SQL and NoSQL database
  • DBMS (Database) - Removed difference between SQL and NoSQL

Fixes Issue

Closes #226

Description:

Added Redis, Oracle and Neo4j databases along with a brief description about each of them. Removed the difference table between SQL and NoSQL database as suggested by the Project Admin/Mentor.
I have also modified the Table of Contents by removing the tag that led to the difference.

Resource URL:

Redis : https://redis.io/
Oracle : https://www.oracle.com/in/database/
Neo4j : https://neo4j.com/product/neo4j-graph-database/

Checklist:

Before submitting your pull request, ensure that you have completed the following tasks:

  • I have carefully reviewed and adhered to the contributing guidelines before creating this pull request.
  • I followed the prescribed PR title template. (Check this if you're adding a resource)

description about each of them. Also added the difference table between SQL and NoSQL database.
Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link
Owner

@jfmartinz jfmartinz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thankjs

@SaileePhal
Copy link
Contributor Author

Sir I guess there has been a small mistake from your side . Sir, I have completed this task and the PR has been successfully merged
and I feel this should be closed whereas you have assigned this to me again .
Also sir not only this put other PR's also have been assigned to me where I have already completed the task and PR has been merges successfully .These are a list of them :

  1. Added contents to DBMS Readme file #236 (comment)
    2 Adding resource #254 (comment)
    3 fixed the bugs  #251 (comment)
    4 Fixing documentation bugs in ds, dbms & ui/ux #329 (comment)
    5 Adding resource under tutorials or courses web dev #326 (comment)
    6 Added more resources under data science tools #325 (comment)
    7 Fixed bugs #315 (comment)
    8 added a link under YouTube Channel in web development file #406 (comment)
    9 Added SQL and NoSQL database and removed table #244 (including this one)

Sir is there any changes you wanted me to do?

@jfmartinz
Copy link
Owner

Gssoc requires it. You can check your email for the announcement, I hate to do it but i dont have choice, for your points to be counted in the leaderboard. I hope this clarifies your question

@SaileePhal
Copy link
Contributor Author

Ohh okk sir my bad, sorry was not aware of this at all since this is my first time .
Thank you so much sir

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

💡 [FEATURE] - Adding some more popular databases under README.md file
2 participants