Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

Update README.md #3171

Closed

Conversation

vikrant-sudo
Copy link
Member

updated readme.md file with out name and github url

What changes does this PR do?

add our name into readme.md file

Screenshots (If any)

Questions

updated readme.md file with out name and github url
@@ -2501,6 +2501,7 @@ A GitHub conflict is when people make changes to the same area or line in a file
- [Vrukshali Torawane](https://github.com/Vrukshali-26)
- [VulcanWM](https://github.com/VulcanWM)
- [Vurugonda Kalyan](https://github.com/kalyan-vurugonda)
- [Vikrant Kumar](https://github.com/vikrant-sudo)
Copy link
Member

Choose a reason for hiding this comment

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

Please add your name in line 2471, after Vikhyath HRS.
Let us know if you need any help doing that. 🙂

Copy link
Member

Choose a reason for hiding this comment

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

Pardon the question, are the names supposed to appear in alphabetical order?

Genuinely curious because I noticed the section above appears to have a repetitive V pattern.

Copy link
Contributor

@schmelto schmelto Oct 7, 2023

Choose a reason for hiding this comment

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

Just saw you found the answer to this #3165 (comment)

In order to maintain consistency and ease of navigation, please arrange your name alphabetically. This includes meticulously considering each letter's position, starting from the first and extending to the second, third, and so on.
Thanks!


Summary: 🛠️ This code needs a fix

Copy link
Member

Choose a reason for hiding this comment

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

Yes I did. Thanks for the further clarification.

@schmelto
Copy link
Contributor

closing due to inactivity feel free to reopen at any time

@schmelto schmelto closed this Oct 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants