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

Format tests #3365

Merged
merged 4 commits into from
Sep 26, 2024
Merged

Format tests #3365

merged 4 commits into from
Sep 26, 2024

Conversation

rmccar
Copy link
Contributor

@rmccar rmccar commented Sep 20, 2024

What is the context of this PR?

Fixes: #3369

Some of the configurations in our macro tests were not formatted correctly. This fixes that.

How to review this PR

Check tests are passing and that they are formatted correctly.

Checklist

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

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

Copy link

netlify bot commented Sep 20, 2024

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

Name Link
🔨 Latest commit ea65a28
🔍 Latest deploy log https://app.netlify.com/sites/ons-design-system-preview/deploys/66ed74b5bf2e7200085bac09
😎 Deploy Preview https://deploy-preview-3365--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.

Copy link

netlify bot commented Sep 20, 2024

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

Name Link
🔨 Latest commit 2f8a932
🔍 Latest deploy log https://app.netlify.com/sites/ons-design-system-preview/deploys/66f3d7d08a0a37000831178b
😎 Deploy Preview https://deploy-preview-3365--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.

@rmccar rmccar added the Tech improvements Tech debt, cleanup, code standardisation etc. label Sep 24, 2024
@rmccar rmccar self-assigned this Sep 24, 2024
@rmccar rmccar marked this pull request as ready for review September 24, 2024 09:24
@rmccar rmccar merged commit 0818e94 into main Sep 26, 2024
42 checks passed
@rmccar rmccar deleted the format-tests branch September 26, 2024 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tech improvements Tech debt, cleanup, code standardisation etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Script tests need formatting
3 participants