Skip to content

Conversation

@sandy4242
Copy link
Contributor

What kind of change does this PR introduce?

probably docs

Issue Number:

Fixes #94

Snapshots/Videos:

image image image

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

@github-actions
Copy link
Contributor

Our Pull Request Approval Process

Thanks for contributing!

Reviewers

Do not assign reviewers. Our Queue Monitors will review your PR and assign them.
When your PR has been assigned reviewers contact them to get your code reviewed and approved via:

  1. comments in this PR

Reviewing Your Code

Your reviewer(s) will have the following roles:

  1. arbitrators of future discussions with other contributors about the validity of your changes
  2. point of contact for evaluating the validity of your work
  3. person who verifies matching issues by others that should be closed.
  4. person who gives general guidance in fixing your tests

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.

Copy link
Contributor

@github-actions github-actions bot left a 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.

@may-tas
Copy link
Owner

may-tas commented Sep 21, 2025

@sandy4242

  1. Update the Folder structure and Folder Details section, current one is outdated.
  2. Upload the Demo Video for web also.
  3. Update the Usage section as we have now added a lot more new functionalities.
  4. Upload new UI screenshots around 8-10 under Font Controls UI.

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?

@sandy4242
Copy link
Contributor Author

@may-tas yes, this pr was to send you the .sh files and everything ...
and to add contributors to the list i think you need to use github action for total automation

@may-tas
Copy link
Owner

may-tas commented Sep 21, 2025

@sandy4242 there is a .github folder present, you can add the necessary GitHub action script.
I'll raise the level, so go ahead and add it.

@sandy4242
Copy link
Contributor Author

@may-tas check the workflow and updated readme.md

@may-tas
Copy link
Owner

may-tas commented Sep 22, 2025

@sandy4242
Explain here in brief how the automation is working, and the role of all the files you have added.

@may-tas
Copy link
Owner

may-tas commented Sep 22, 2025

@sandy4242 Please make sure all the screenshots are of similar size
Screenshot 2025-09-22 at 7 00 24 PM

@sandy4242
Copy link
Contributor Author

sandy4242 commented Sep 22, 2025

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 all-contributors add $username code and adds the newly contributor to the readme, the .all-contributorsrc file gets all the contributors from the git logs/history, the add-all-contributors.sh files add the contributors to the readme and the package.json has the all-contributor cli installed.
If this pr gets merged, i hope the workflow would proceed in the similar fashion and add me to the readme too as a contributor

@may-tas
Copy link
Owner

may-tas commented Sep 22, 2025

@sandy4242 Okay , I hope it works as expected.

@may-tas may-tas merged commit 1e59241 into may-tas:main Sep 22, 2025
@may-tas
Copy link
Owner

may-tas commented Sep 22, 2025

@may-tas
Copy link
Owner

may-tas commented Sep 22, 2025

@sandy4242 I have reopened the issue, please raise a PR to fix the workflow.

@sandy4242
Copy link
Contributor Author

@may-tas let me see check the workflow

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 Request: Improving & Updating README.md

2 participants