Skip to content

Conversation

@mattshaner
Copy link

Elements Default PR Template

In general, make sure you have: (check the boxes to acknowledge you've followed this template)

Other Available PR Templates:

@mattshaner mattshaner requested a review from a team as a code owner October 28, 2025 17:24
@mattshaner mattshaner requested a review from lukasikp October 28, 2025 17:24
@netlify
Copy link

netlify bot commented Oct 28, 2025

Deploy Preview for stoplight-elements ready!

Name Link
🔨 Latest commit d0bc434
🔍 Latest deploy log https://app.netlify.com/projects/stoplight-elements/deploys/6900fc51305f6b00087ac233
😎 Deploy Preview https://deploy-preview-2840--stoplight-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Oct 28, 2025

Deploy Preview for stoplight-elements-demo ready!

Name Link
🔨 Latest commit d0bc434
🔍 Latest deploy log https://app.netlify.com/projects/stoplight-elements-demo/deploys/6900fc5167957c000821288b
😎 Deploy Preview https://deploy-preview-2840--stoplight-elements-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

on:
schedule:
- cron: 0 3 * * 1
# # 'filters' was not transformed because there is no suitable equivalent in GitHub Actions
Copy link
Contributor

Choose a reason for hiding this comment

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

why is this comment here? i'm assuming this was just run through an llm, but what do we lose/gain by not using the filters from circle ci?

Copy link
Author

Choose a reason for hiding this comment

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

This was run through a github actions importer tool from microsoft. The filters I believe were to enforce semver on the tags. I can fix that I found a solution using github filters.

Copy link
Author

@mattshaner mattshaner Oct 29, 2025

Choose a reason for hiding this comment

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

I looked at the orginal config file this is the original filters code:

filters:
branches:
only:
- main
Scheduled jobs run only on the last commit of the default branch so a filter shouldn't be needed.

- name: Example - use local builds
run: yarn copy:${{ env.example-name }}
working-directory: "/mnt/ramdisk/project"
- name: Example - Install dependencies
Copy link
Contributor

Choose a reason for hiding this comment

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

why are the names sometimes Example - and sometimes not?

Copy link
Author

Choose a reason for hiding this comment

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

This is from the tool I used. I am cross referencing everything the tool did versus the circleci config file. I review this more before I merge.

@ben-smartbear ben-smartbear self-requested a review November 10, 2025 14:59
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.

3 participants