My First Open Source Contribution Got Merged! 🎉 — cucumber-reporting #1333

Contributing
1

Hey everyone!

I’m excited to share that my first open source pull request just got merged into damiansczepanik/cucumber-reporting!

What the feature does: The PR adds a “Total” line to the trend statistics charts in cucumber-reporting. Previously the charts only showed Failed and Passed lines, making it hard to get a sense of the overall project scale at a glance. The Total line now appears as a separate axis on the right side of the chart, styled in gray with its own scale so it doesn’t interfere with the stacked Failed/Passed visualization.

What I learned from this experience:

  • How to navigate a real open source codebase and understand an existing PR pattern before writing code

  • How to handle code review feedback — the reviewer requested multiple rounds of changes and I had to carefully address each one

  • The importance of keeping PRs focused — I accidentally included an unrelated change in array.js.vm which the reviewer immediately caught and flagged

  • How to use git commit --amend and git push --force to keep a clean single commit history

  • How to sync a fork with upstream using git pull upstream master

The review process was humbling but educational. The reviewer damiansczepanik was thorough and patient. At one point he asked if I was even going to complete the PR after a period of inactivity due to my exams. That was a wake up call to be more communicative in open source.

PR link: https://github.com/damianszczepanik/cucumber-reporting/pull/1333

If you’re thinking about making your first open source contribution, just go for it. The process of working through real feedback on a real codebase teaches you things no tutorial can.

2

Quick update — PR #1338 just got merged too! :tada:

This one was a cleanup PR: removing an unneeded setTrendsStatsFile line and fixing indentation left over from my previous work on #1333. Small change, but it felt good to follow up and keep the codebase tidy rather than leaving loose ends.

PR link: cleanup: remove unneeded setTrendsStatsFile line and fix indentation by kushagra4321gkp-droid · Pull Request #1338 · damianszczepanik/cucumber-reporting · GitHub

Two PRs merged in the same project within days of each other — definitely didn’t expect that when I started. Each one taught me something different, and I’m genuinely enjoying the process of contributing to a real codebase. Looking forward to finding the next issue to work on!