Skip to content

Add build plugin artifact to releases branch #3232

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: releases
Choose a base branch
from
Open

Add build plugin artifact to releases branch #3232

wants to merge 2 commits into from

Conversation

tzahgr
Copy link
Contributor

@tzahgr tzahgr commented May 23, 2025

David Evbodaghe added 2 commits May 23, 2025 15:06
Summary:
[TEST PR]

Pull Request resolved: #3052

Reviewed By: SayanPandey

Differential Revision:
D74307699

Privacy Context Container: L1313315

Pulled By: devbodaghe

fbshipit-source-id: e7f790969e5ca2acd160b8d81f4460dd334b5d4e
Summary:
## Description

This PR improves the GitHub Actions workflow for building plugin artifacts by implementing comment management functionality. Instead of creating a new comment for each build, the workflow now updates an existing comment if one exists, reducing PR noise and improving readability.

### Type of change

- Syntax change (non-breaking change which fixes code modularity, linting or phpcs issues)

## Checklist

- [x] I have commented my code, particularly in hard-to-understand areas.
- [x] I have confirmed that my changes do not introduce any new PHPCS warnings or errors.
- [x] I have checked plugin debug logs that my changes do not introduce any new PHP warnings or FATAL errors.
- [x] I followed general Pull Request best practices. Meta employees to follow this [wiki](https://fburl.com/wiki/2cgfduwc).
- [x] I have added tests (if necessary) and all the new and existing unit tests pass locally with my changes.
- [x] I have completed dogfooding and QA testing, or I have conducted thorough due diligence to ensure that it does not break existing functionality.
- [x] I have updated or requested update to plugin documentations (if necessary). Meta employees to follow this [wiki](https://fburl.com/wiki/nhx73tgs).

## Changelog entry

Improve GitHub Actions workflow to update existing build comments instead of creating new ones

Pull Request resolved: #3218

Test Plan:
1. Created a test PR to verify the workflow behavior
2. Made multiple commits to trigger the workflow
3. Verified that:
   - First run creates a new comment
   - Subsequent runs update the existing comment instead of creating new ones
   - Comment content is properly updated with latest build information
   - Error handling works as expected

## Screenshots
### Before
<img width="1065" alt="Screenshot 2025-05-19 at 19 57 00" src="https://github.com/user-attachments/assets/afd2d392-b5ac-4d69-963a-552456a5b18c" />

### After
<img width="1074" alt="Screenshot 2025-05-19 at 19 56 37" src="https://github.com/user-attachments/assets/e4bcdc5a-d6ed-480e-972c-0ef1976364de" />

Reviewed By: gurtejrehal

Differential Revision:
D75014048

Privacy Context Container: L1313315

Pulled By: devbodaghe

fbshipit-source-id: fa006f51a0bfd1176d908286eb528513a2e0403e
Copy link
Contributor

📦 Latest Plugin Build

Built at: 2025-05-23T14:12:18.652Z
Commit: a7794a7
Size: 972K

Download: Click here to download the plugin

To download: Click the link above → Scroll to bottom → Download "facebook-for-woocommerce" artifact

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants