-
Notifications
You must be signed in to change notification settings - Fork 35
Updating markup file with automation #96
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
Our Pull Request Approval ProcessThanks for contributing! ReviewersDo not assign reviewers. Our Queue Monitors will review your PR and assign them.
Reviewing Your CodeYour reviewer(s) will have the following roles:
Other🎯 Please be considerate of our volunteers' time. Contacting the person who assigned the reviewers is not advised unless they ask for your input. Do not @ the person who did the assignment otherwise. |
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.
🎉 Congratulations on making your first PR! If you haven't already, check out our Contributing Guidelines and README to ensure that you are following our guidelines for contributing.
also kindly mention how the automation is working here in comments, and the role of all the files you have added. Also once this PR is merged would you be added automatically in the contributor section? |
|
@may-tas yes, this pr was to send you the .sh files and everything ... |
|
@sandy4242 there is a .github folder present, you can add the necessary GitHub action script. |
|
@may-tas check the workflow and updated readme.md |
|
@sandy4242 |
|
@sandy4242 Please make sure all the screenshots are of similar size |
|
so the .yml file will check for new merges from new users and add their name and profile on the readme which would be approved by you and if any contributor is missed it does a routine check on weekly basis, the allContributon action prints the msg |
|
@sandy4242 Okay , I hope it works as expected. |
|
@sandy4242 the script is failing |
|
@sandy4242 I have reopened the issue, please raise a PR to fix the workflow. |
|
@may-tas let me see check the workflow |

What kind of change does this PR introduce?
probably docs
Issue Number:
Fixes #94
Snapshots/Videos:
Summary
Adds contributors to the readme.md file
Closes #94
Does this PR introduce a breaking change?
no
Other information
non
Have you read the contributing guide , README.md , code of conduct?
yes