Add monthly featured developer showcase by Aashish-Jha-11 · Pull Request #124 · john-bampton/john-bampton.github.io

Aashish-Jha-11 · 2025-12-17T13:58:24Z

Highlights top GitHub developer each month based on engagement (followers, stars, repos, sponsors, activity). Features beautiful gradient banner with user stats and top languages. Auto-updates when running fetch.py.

Current featured: Linus Torvalds (engagement score: 74.97)

What's included:

🎯 Smart engagement scoring algorithm (40% followers, 30% stars, 15% repos, 10% sponsors, 5% recent activity)
🎨 Beautiful gradient banner with glassmorphism effects
📱 Fully responsive design (mobile, tablet, desktop)
🔄 Automatically updates when running fetch.py
⚡ Quick manual generation via generate_featured.py

Resolves #122

Highlights top GitHub developer each month based on engagement (followers, stars, repos, sponsors, activity). Features beautiful gradient banner with user stats and top languages. Auto-updates when running fetch.py. Current featured: Linus Torvalds (engagement score: 74.97) Resolves john-bampton#122

gemini-code-assist · 2025-12-17T13:58:42Z

Summary of Changes

Hello @Aashish-Jha-11, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request introduces a significant new feature to showcase a 'Featured Developer of the Month' on the platform. It encompasses both backend logic for selecting the developer based on a comprehensive engagement score and frontend UI/UX development to present this information attractively and responsively. The goal is to celebrate highly engaged community members and provide an engaging visual element to the site.

Highlights

Monthly Featured Developer: Introduces a new 'Featured Developer of the Month' section to highlight top GitHub developers based on engagement.
Engagement Scoring Algorithm: Implemented a smart algorithm in Python to calculate a developer's engagement score, considering followers, stars, repositories, sponsors, and recent activity.
Dynamic UI Integration: The frontend now dynamically loads and displays the featured developer's information, including avatar, name, login, location, key stats, and top languages, with a visually appealing gradient banner and glassmorphism effects.
Automated & Manual Generation: The featured developer data automatically updates when fetch.py is run, and a new utility script generate_featured.py allows for quick manual generation.
Responsive Design: The new featured developer section is fully responsive, ensuring optimal display across mobile, tablet, and desktop devices.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

gemini-code-assist

Code Review

This pull request introduces a great new feature: a monthly featured developer showcase. The implementation spans the frontend and backend, with new UI components, styling, and Python scripts for data processing. My review identifies a critical bug in fetch.py due to a duplicated function, which needs immediate attention. I've also pointed out a high-priority maintainability issue concerning duplicated logic between two Python scripts. Other comments focus on improving code quality and maintainability through better error handling, code structure, and adherence to standard coding practices. The frontend work is generally solid, with good considerations for performance and security.

BaseMax · 2025-12-17T15:20:07Z

@Aashish-Jha-11 shows me a screenshot once everything is ready.

Aashish-Jha-11 · 2025-12-17T15:28:53Z

@BaseMax

BaseMax · 2025-12-17T15:45:53Z

Descriptions are not readable well:

BaseMax · 2025-12-17T15:46:43Z

‎docs/featured.json‎ should be without ident, so ident=0.

And I think we need to merge both json into one as in this case we will have some delay to load this big featured developer section and will make scroll changes for end-user.

- Increased stat label font size and contrast for better readability - Removed indentation from featured.json (compact format) - Updated both fetch.py and generate_featured.py

BaseMax · 2025-12-17T16:01:06Z

And I think we need to merge both json into one as in this case we will have some delay to load this big featured developer section and will make scroll changes for end-user.

Let me know your feedback about this.

Aashish-Jha-11 · 2025-12-17T16:15:17Z

@BaseMax Sir! I think Keeping JSONs separate is better:

Both load in parallel Promise.all & featured.json
No scroll jumping - featured section loads immediately
Better cache management - can update featured monthly without invalidating main cache

BaseMax · 2025-12-17T16:26:41Z

Better cache management - can update featured monthly without invalidating main cache

Did you handled it in manifest.json or service-worker.js file?

Aashish-Jha-11 · 2025-12-17T16:52:37Z

@BaseMax Not explicitly - I meant browser-level caching. Separate files = independent cache entries. When featured.jsonupdates monthly, browsers can keep cached users.json (saves 640KB re-download).

Should we add explicit cache handling in service-worker.js? We could set different cache durations (1 month for featured.json vs longer for users.json).

BaseMax · 2025-12-17T18:27:29Z

Should we add explicit cache handling in service-worker.js? We could set different cache durations (1 month for featured.json vs longer for users.json).

Yes.

BaseMax · 2025-12-17T18:27:46Z

1 months is too mcuh, lets set 1 week for featured, and 1 day for users.json.

- featured.json: 1 week cache - users.json: 1 day cache - Adds timestamp tracking for cache validation

BaseMax · 2025-12-21T21:24:11Z

"Due to the large number of changes in this pull request, only one file is being shown at a time." something is wrong in this PR.

BaseMax · 2025-12-21T21:29:06Z

This PR is open for a while, please finalize and do the task sooner so we can merge and have it up/ready.

Thanks, and Happy Yalda Night.

Aashish-Jha-11 · 2025-12-22T08:35:15Z

Hi @BaseMax Sir, thanks for the patience. The large diff is from prettier formatting bootstrap vendor files and users.json data file.

Current blockers:
Super-linter still failing - could you share which specific linter errors are blocking?
New merge conflicts appeared with main branch

I can resolve both quickly once I know the specific linter issues. The core feature code (~300 lines) is complete and tested.

Happy Yalda Night! 🎉

jbampton · 2025-12-22T11:56:54Z

This branch has conflicts that must be resolved

BaseMax · 2025-12-25T06:21:17Z

This branch has conflicts that must be resolved

BaseMax · 2025-12-25T06:21:31Z

We need to close and merge it faster. @Aashish-Jha-11 please do action.

Aashish-Jha-11 · 2025-12-25T06:36:43Z

Hi @BaseMax, understood - I'll act immediately Please tell me your opinion then only i would be able to proceed please.
Quick context: The PR was working at stage 1, but super-linter failures forced formatting changes that ballooned the diff (bootstrap vendor files, etc.).

Two options to close this fast:

Revert to working stage 1 (before lint changes) - this auto-fixes conflicts and gives clean feature-only code
Fix current conflicts and deal with large diff from formatters
Which approach do you prefer? I can execute either within hours.

BaseMax · 2025-12-25T07:50:59Z

Please make sure you pull from latest version of main and fix all conflicts. Complete your PR, I want to merge and develop it more.

Best

BaseMax · 2025-12-25T07:52:01Z

Which approach do you prefer?

I already updated pre-commit on the main branch, and it will run linters, etc so don't change formats of code at all. pre-commit will take care of it for now.

deepsource-io · 2025-12-25T07:57:41Z

Here's the code health analysis summary for commits c5b753d..5a5343f. View details on DeepSource ↗.

Analysis Summary

Analyzer	Status	Summary	Link
Secrets	✅ Success		View Check ↗
Python	❌ Failure	❗ 13 occurences introduced 🎯 2 occurences resolved	View Check ↗
JavaScript	❌ Failure	❗ 787 occurences introduced 🎯 824 occurences resolved	View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

BaseMax · 2025-12-25T08:12:27Z

This branch has conflicts that must be resolved

Aashish-Jha-11 · 2025-12-25T08:14:12Z

Hi @BaseMax,

Resolved merge conflicts and fixed all critical issues in our feature code. Latest commit: 4e08e3b

Fixed:
✅ Python: All logging f-strings converted to parameter substitution, jinja2 autoescape enabled
✅ JavaScript: Strict equality (=== instead of ==) in our feature code

Still failing (DeepSource):
The remaining issues are in Bootstrap vendor files (jquery.min.js, bootstrap.js, popper.min.js) that were reformatted by prettier:

JS-0125 (Undeclared variables) - jQuery/Bootstrap patterns
JS-0128 (Unused variables) - Vendor code
JS-0103 (IIFE wrapping) - jQuery patterns
Others - All in vendor libraries
Our feature code (script.js, service-worker.js, fetch.py, render.py) is clean and follows best practices.

Options:

Exclude vendor files from DeepSource analysis
Revert vendor file formatting (keep them minified)
Merge as-is since issues are in external dependencies
Ready for your decision.

BaseMax · 2025-12-25T08:15:43Z

docs/bootstrap/css/bootstrap-grid.css‎

all bootstrap assets files should be minify.

BaseMax · 2025-12-25T08:16:01Z

If we have both .min.css and nonmin.css so please remove nonmin and just keep using .min.css or .min.js.

BaseMax · 2025-12-25T08:17:32Z

you should keep ‎docs/featured.json‎ file minified.

BaseMax · 2025-12-25T08:17:49Z

you should keep docs/manifest.json‎ file minified. or maybe this comes from pre-commit or something?

BaseMax · 2025-12-25T08:19:46Z

I will not merge it, you damaged to my recent updates such as docs/service-worker.js‎, keep my version which is better.

BaseMax · 2025-12-25T08:20:37Z

You removed some code from html which are good to keep.

such as

                <option value="followers-growth-desc">
                  Trending (Followers ↑)
                </option>
                <option value="followers-growth-asc">
                  Trending (Followers ↓)
                </option>

and

            <button class="btn-apply" onclick="exportFilteredJSON()">
              Export JSON
            </button>
            <button class="btn-apply" onclick="exportFilteredCSV()">
              Export CSV
            </button>

BaseMax · 2025-12-25T08:24:52Z

You have removed most codes in fetch.py and render.py such as:

You removed other codes:

BaseMax · 2025-12-25T08:27:49Z

If it's hard for you to check/fix conflicts with pay attention to details. i suggest you to close this pr and open a new one and just copy paste your important/new changes

BaseMax · 2025-12-25T08:38:24Z

I have removed nonmin css and js bootstrap files in main branch.

Aashish-Jha-11 · 2025-12-25T14:34:28Z

Closing this PR due to merge conflict damage. Replaced with clean implementation in #187

gemini-code-assist Bot reviewed Dec 17, 2025

View reviewed changes

Comment thread fetch.py Outdated

Comment thread generate_featured.py

Comment thread docs/script.js Outdated

Comment thread docs/script.js Outdated

Comment thread fetch.py Outdated

Comment thread fetch.py

Comment thread generate_featured.py

jbampton self-assigned this Dec 17, 2025

jbampton added enhancement New feature or request help wanted Extra attention is needed hacktoberfest labels Dec 17, 2025

jbampton added this to Turbo Dec 17, 2025

jbampton moved this to In Progress in Turbo Dec 17, 2025

jbampton added this to the December 2025 milestone Dec 17, 2025

jbampton added javascript python css layouts json labels Dec 17, 2025

Fix: Improve label readability and remove JSON indentation

5912f7c

- Increased stat label font size and contrast for better readability - Removed indentation from featured.json (compact format) - Updated both fetch.py and generate_featured.py

Fix: Make stat labels fully visible with stronger styling

129a53d

Merge branch 'main' into feature/monthly-featured-developer

5edf300

Fix: Apply black and isort formatting to Python files

083ffff

Add cache duration handling for JSON files

a699a76

- featured.json: 1 week cache - users.json: 1 day cache - Adds timestamp tracking for cache validation

Merge upstream/main - resolve conflicts, keep feature code

a0e254d

Aashish-Jha-11 added 4 commits December 25, 2025 08:00

Fix DeepSource issues: enable jinja2 autoescape, use logging params

d4a08db

Fix Python syntax errors: properly convert all logging f-strings

ddeb7eb

Fix final 2 Python logging f-strings

3b06805

Fix JS linting: replace == with === for strict equality

4e08e3b

Merge upstream/main: add sitemap and safe_path functions

5a5343f

Aashish-Jha-11 closed this Dec 25, 2025

github-project-automation Bot moved this from In Progress to Done in Turbo Dec 25, 2025

Uh oh!

Conversation

Aashish-Jha-11 commented Dec 17, 2025

Uh oh!

gemini-code-assist Bot commented Dec 17, 2025

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

BaseMax commented Dec 17, 2025

Uh oh!

Aashish-Jha-11 commented Dec 17, 2025

Uh oh!

BaseMax commented Dec 17, 2025

Uh oh!

BaseMax commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

BaseMax commented Dec 17, 2025

Uh oh!

Aashish-Jha-11 commented Dec 17, 2025

Uh oh!

BaseMax commented Dec 17, 2025

Uh oh!

Aashish-Jha-11 commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

BaseMax commented Dec 17, 2025

Uh oh!

BaseMax commented Dec 17, 2025

Uh oh!

BaseMax commented Dec 21, 2025

Uh oh!

BaseMax commented Dec 21, 2025

Uh oh!

Aashish-Jha-11 commented Dec 22, 2025

Uh oh!

jbampton commented Dec 22, 2025

Uh oh!

BaseMax commented Dec 25, 2025

Uh oh!

BaseMax commented Dec 25, 2025

Uh oh!

Aashish-Jha-11 commented Dec 25, 2025

Uh oh!

BaseMax commented Dec 25, 2025

Uh oh!

BaseMax commented Dec 25, 2025

Uh oh!

deepsource-io Bot commented Dec 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Analysis Summary

Uh oh!

BaseMax commented Dec 25, 2025

Uh oh!

Aashish-Jha-11 commented Dec 25, 2025

Uh oh!

BaseMax commented Dec 25, 2025

Uh oh!

BaseMax commented Dec 25, 2025

Uh oh!

BaseMax commented Dec 25, 2025

Uh oh!

BaseMax commented Dec 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

BaseMax commented Dec 25, 2025

Uh oh!

BaseMax commented Dec 25, 2025

BaseMax commented Dec 17, 2025 •

edited

Loading

Aashish-Jha-11 commented Dec 17, 2025 •

edited

Loading

deepsource-io Bot commented Dec 25, 2025 •

edited

Loading

BaseMax commented Dec 25, 2025 •

edited

Loading