Skip to content
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

Updated backstop js config #3149

Merged
merged 4 commits into from
Apr 19, 2024
Merged

Updated backstop js config #3149

merged 4 commits into from
Apr 19, 2024

Conversation

alessioventuriniAND
Copy link
Contributor

@alessioventuriniAND alessioventuriniAND commented Apr 17, 2024

What is the context of this PR?

Backstop js current match threshold is 0.1 which is not picking up all the changes that we have recently introduce in the radios component. I have now increase the accuracy by bringing this value down to 0.00. As you can see all the recent changes have now been picked up by backstop.js

How to review this PR

Confirm that the reference images that have been updated are expected to be updated.

Checklist

This needs to be completed by the person raising the PR.

  • I have selected the correct Assignee
  • I have linked the correct Issue

@alessioventuriniAND alessioventuriniAND added the Bug Something isn't working label Apr 17, 2024
@alessioventuriniAND alessioventuriniAND requested a review from a team April 17, 2024 14:20
@alessioventuriniAND alessioventuriniAND self-assigned this Apr 17, 2024
Copy link

netlify bot commented Apr 17, 2024

Deploy Preview for ons-design-system-preview ready!

Name Link
🔨 Latest commit 5fab81e
🔍 Latest deploy log https://app.netlify.com/sites/ons-design-system-preview/deploys/6620d9619262eb00086fd5b8
😎 Deploy Preview https://deploy-preview-3149--ons-design-system-preview.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 site configuration.

@adi-unni adi-unni merged commit 17098be into main Apr 19, 2024
9 checks passed
@adi-unni adi-unni deleted the fix/backstopjs-config branch April 19, 2024 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants