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

Updated README #34

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Updated README #34

wants to merge 4 commits into from

Conversation

Agnik7
Copy link

@Agnik7 Agnik7 commented Aug 27, 2023

Updated the information about Introduction to AI in the README file to make it more easier to grasp.

@Agnik7
Copy link
Author

Agnik7 commented Aug 27, 2023

@microsoft-github-policy-service agree

Copy link

@TBTS-2011 TBTS-2011 left a comment

Choose a reason for hiding this comment

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

Great! The only perk that could need to be fixed is that a README file is not supposed to be fifty lines long, and removing the first line of this file, which is already fitting is not a great idea. Other than that, no problems!

@Agnik7
Copy link
Author

Agnik7 commented Aug 29, 2023

I could not bring the number of lines below 50, however, I have tried to reduce it to 65 lines, while keeping the material intact.

A notable change I did, is that, I have removed the details from the Contributions, as I observed that a markdown file exists that contains the exact same content about the contribution, so I have just provided the link to that file.

@pavans2011 Kindly review this.

Copy link

@TBTS-2011 TBTS-2011 left a comment

Choose a reason for hiding this comment

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

Great job pal! Brilliant work, even though it's just a README file, your imperovement counts. Better than the previous change log by the way!

@Agnik7
Copy link
Author

Agnik7 commented Aug 29, 2023

Thanks a lot @pavans2011 for taking out the time to review this small contribution.

@TBTS-2011
Copy link

NP pal! Always available to help a hand!

@TBTS-2011 TBTS-2011 mentioned this pull request Oct 12, 2023
Copy link

@TBTS-2011 TBTS-2011 left a comment

Choose a reason for hiding this comment

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

I think I will approve this commit because it doesn't have any conflicts with the main branch.

Copy link

@RahulVadisetty91 RahulVadisetty91 left a comment

Choose a reason for hiding this comment

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

Ensure the code block is complete and properly formatted.

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

Successfully merging this pull request may close these issues.

4 participants