Readme contributor wall#84
Conversation
- Applied the specified contributor design. - Applied marked down table for contributors display. - Auto contributor updation
Signed-off-by: AdarshSingh-25 <adarshsinghofficial25@gmail.com>
Removed auto-update notice for contributors. Signed-off-by: AdarshSingh-25 <adarshsinghofficial25@gmail.com>
✅ Logic Verified!The Automated Logic Suite has confirmed that 1 is still 1.
You are safe to merge this chaos. |
There was a problem hiding this comment.
👋 Welcome to is-one-one, @AdarshSingh-25!
Thank you for submitting your first pull request! We really appreciate your time and effort in contributing to the project.
Our automated test suite will run shortly to verify your contribution. In the meantime, please review our Contributing Guidelines to ensure your PR matches our repository requirements. Happy coding! 🧠
|
Hello @Luffy-456, |
Luffy-456
left a comment
There was a problem hiding this comment.
lgtm,
Can be merged
Nice work considering its your first open source issue
There was a problem hiding this comment.
Hi @AdarshSingh-25, @Luffy-456 Thank for contributing. This automation good but before we merge this in, I have a request regarding how the contributor grid visually looks on the readme.
Right now, using the classic HTML <table> (<tr> and <td> cells) creates a rigid, boxy layout that leaves a lot of weird, empty square padding around the profile cards depending on the browser window width. It also strips out horizontal flow if there are any newlines inside the string blocks.
To make the layout fluid and modern, could we drop the HTML table tags completely and switch to a modern CSS Flexbox Grid? or you can try what you feel looks better for readme and share the new screenshot of fix
Signed-off-by: AdarshSingh-25 <adarshsinghofficial25@gmail.com>
itsdakshjain
left a comment
There was a problem hiding this comment.
Share the new screenshot too @AdarshSingh-25
Signed-off-by: AdarshSingh-25 <adarshsinghofficial25@gmail.com>
|
Hi @itsdakshjain |
|
By screen shot can u show me like what now 20 or few contributor in readme all together looks like and I didn't mean to remove table I just suggested and like what you feels and look good by trying things , as in your original pr description screenshot was good but just empty space if we can reduce that and somehow make it look cool One more thing I noticed you making test branch in your fork. Make changes there so we can see how the new section looks like after testing |
|
@itsdakshjain, okay i will do it |

Description
Add an automated contibutor wall that updates the README based on repository contributors.
contributor-wallto fetch contributors automatically.build-contributor-wallto generate contributor wall HTML.Closes #37
Type of Change
Checklist
Overengineering Verification
Validation
pytestpasses locally with no failuresTraceability
General
Screenshots (if applicable)
Additional Notes
To reflect the changes immediately, manually trigger the workflow.