Skip to content

Readme contributor wall#84

Open
AdarshSingh-25 wants to merge 6 commits into
Py-Contributors:mainfrom
AdarshSingh-25:readme-contributor-wall
Open

Readme contributor wall#84
AdarshSingh-25 wants to merge 6 commits into
Py-Contributors:mainfrom
AdarshSingh-25:readme-contributor-wall

Conversation

@AdarshSingh-25

Copy link
Copy Markdown

Description

Add an automated contibutor wall that updates the README based on repository contributors.

  • Added a workflow contributor-wall to fetch contributors automatically.
  • Added a scripts build-contributor-wall to generate contributor wall HTML.
  • Supports automatic update on commits and PRs.

Closes #37


Type of Change

  • feat
  • fix
  • docs
  • refactor
  • chore
  • accidental genius

Checklist

Overengineering Verification

  • I have introduced at least one unnecessary abstraction layer
  • The solution is more complex than the problem warrants
  • A reasonable engineer would question at least one design decision
  • I have named something in a way that requires further explanation

Validation

  • pytest passes locally with no failures
  • New functionality is covered by tests
  • I have not deleted tests to make CI green
  • Documentation has been updated if needed

Traceability

  • This PR references a related issue
  • The branch name vaguely reflects the chaos introduced

General

  • Future contributors may question my decisions, but not my confidence
  • Code has been self-reviewed
  • No secrets, credentials, or hardcoded paths were smuggled in
  • I have read the CONTRIBUTING.md guidelines

Screenshots (if applicable)

image

Additional Notes

To reflect the changes immediately, manually trigger the workflow.


"Any fool can write code that a computer can understand. Good programmers write code that humans can understand — but we don't do that here."

- 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>
@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

✅ Logic Verified!

The Automated Logic Suite has confirmed that 1 is still 1.

  • Environment: Ubuntu-latest
  • Status: Mathematical Sanity Confirmed

You are safe to merge this chaos.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👋 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! 🧠

@AdarshSingh-25

Copy link
Copy Markdown
Author

Hello @Luffy-456,
please review this PR.

@Luffy-456 Luffy-456 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm,
Can be merged
Nice work considering its your first open source issue

@itsdakshjain itsdakshjain left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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>
@Py-Contributors Py-Contributors deleted a comment from github-actions Bot Jun 4, 2026
@Py-Contributors Py-Contributors deleted a comment from github-actions Bot Jun 4, 2026

@itsdakshjain itsdakshjain left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Share the new screenshot too @AdarshSingh-25

Signed-off-by: AdarshSingh-25 <adarshsinghofficial25@gmail.com>
@AdarshSingh-25

Copy link
Copy Markdown
Author

Hi @itsdakshjain
I looked into this and ran into a GitHub README limitation.
The original table layout can show avatars, usernames, and commit counts reliably, but it creates the rigid grid you mentioned. To get a modern flowing layout, we'd need Flexbox/Grid styling, which GitHub strips from README rendering.
I switch to a flowing avatar cloud (no table), but in the README that would realistically only show the profile pictures as clickable links. Usernames and commit counts won't render.
image
but i can create a flowing layout which shows name and commits too. but it will only make a link in readme, it won't show contributors in readme directly

@Py-Contributors Py-Contributors deleted a comment from github-actions Bot Jun 4, 2026
@itsdakshjain

Copy link
Copy Markdown
Member

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
Or if we remove table and as the commits you did what do you mean by it won't show contributors in readme directly
So you mean the commits number ? As I guess our priority is to just make contributors section not exact commit if that's a hindrance

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

@AdarshSingh-25

Copy link
Copy Markdown
Author

@itsdakshjain, okay i will do it

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.

Docs: Add a visual Contributor Gallery to README

3 participants