Skip to content

Conversation

@sheikhlimon
Copy link
Contributor

@sheikhlimon sheikhlimon commented Oct 23, 2025

Description

The PR metadata sync workflow was running on every PR edit, including when users check/uncheck checkboxes in the PR description. This caused:

  • Duplicate metadata sync comments
  • Unnecessary workflow runs
  • Noise in PR activity

Fixes #1015

This is a follow-up to which was previously closed with a partial fix. The original fix attempted to detect checkbox-only changes, but the root cause is that the edited trigger itself is unnecessary for this workflow.

Solution

Removed the edited event trigger from the workflow. The workflow now only runs on:

  • opened - When PR is first created
  • synchronize - When new commits are pushed
  • reopened - When PR is reopened

I make a change and metadata gets created again

@vercel
Copy link
Contributor

vercel bot commented Oct 23, 2025

@sheikhlimon is attempting to deploy a commit to the recode Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs.

In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊

Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv.

We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰

🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨

📚Your perks for contribution to this community 👇🏻

  1. Get free Consultation use code recode50 to get free: Mentorship for free.

  2. Get the Ebook for free use code recode at checkout: Data Science cheatsheet for Beginners.

  3. Check out this weekly Newsletter: Sanjay's Newsletter.

If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@github-actions github-actions bot added hacktoberfest-accepted recode this is label for leaderboard level 1 10 points labels Oct 23, 2025
@github-actions
Copy link

✅ Synchronized metadata from Issue #1015:

  • Labels: level 1, recode, hacktoberfest-accepted
  • Milestone: recode:launch 3.0

@github-actions github-actions bot added this to the recode:launch 3.0 milestone Oct 23, 2025
@iitzIrFan iitzIrFan added the under review Review under the maintainers or the admins label Oct 23, 2025
@iitzIrFan iitzIrFan moved this to In Progress in @recode-web Oct 23, 2025
@iitzIrFan
Copy link
Member

@sheikhlimon So what will be the expected behaviour after this update in the sync feature?

@sheikhlimon
Copy link
Contributor Author

sheikhlimon commented Oct 23, 2025

@sheikhlimon So what will be the expected behaviour after this update in the sync feature?

See the other PR when they are edited, like most of them check the box after creating the PR... which makes the metadata basically being created like the amount of times it's being edited, that's sort of unnecessary

Edit: I made the change in the PR body, which recreated the metadata below. However, if that's what's expected, that's fine. It's mainly for re-adding Issue No., but people rarely do that.

@github-actions
Copy link

✅ Synchronized metadata from Issue #1015:

  • Labels: level 1, recode, hacktoberfest-accepted
  • Milestone: recode:launch 3.0

Copy link
Member

@iitzIrFan iitzIrFan left a comment

Choose a reason for hiding this comment

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

LGTM! thanks for the pr @sheikhlimon

@vercel
Copy link
Contributor

vercel bot commented Oct 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
recode-website Ready Ready Preview Comment Oct 23, 2025 9:25pm

@sanjay-kv sanjay-kv merged commit ab55f66 into recodehive:main Oct 23, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in @recode-web Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted level 1 10 points recode this is label for leaderboard under review Review under the maintainers or the admins

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

🐞[Bug]: Prevent workflow from running on checkbox toggles in PR body

3 participants