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

Fix(web-twig): Fix pass custom classnames to the Spinner component #1099

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

dlouhak
Copy link
Contributor

@dlouhak dlouhak commented Oct 19, 2023

No description provided.

@dlouhak dlouhak added the bug Something isn't working label Oct 19, 2023
@dlouhak dlouhak self-assigned this Oct 19, 2023
@netlify
Copy link

netlify bot commented Oct 19, 2023

Deploy Preview for spirit-design-system-react canceled.

Name Link
🔨 Latest commit de57f0f
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-react/deploys/65312859af3cf40008acca1d

@netlify
Copy link

netlify bot commented Oct 19, 2023

Deploy Preview for spirit-design-system-validations canceled.

Name Link
🔨 Latest commit de57f0f
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-validations/deploys/653128598e43280008d2ee55

@netlify
Copy link

netlify bot commented Oct 19, 2023

Deploy Preview for spirit-design-system-storybook canceled.

Name Link
🔨 Latest commit de57f0f
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-storybook/deploys/65312859fec406000803543b

@netlify
Copy link

netlify bot commented Oct 19, 2023

Deploy Preview for spirit-design-system-demo canceled.

Name Link
🔨 Latest commit de57f0f
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-demo/deploys/653128596b3c880008841beb

@nx-cloud
Copy link

nx-cloud bot commented Oct 19, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit de57f0f. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 3 targets

Sent with 💌 from NxCloud.

@github-actions
Copy link
Contributor

Coverage Status

coverage: 96.46%. remained the same when pulling 2e93659 on bugfix/spinner-classnames into 0cb30d5 on main.

@crishpeen
Copy link
Member

No description provided? Can you please explain how this happened so we can prevent it in the future?

@dlouhak dlouhak force-pushed the bugfix/spinner-classnames branch from 2e93659 to de57f0f Compare October 19, 2023 13:00
@dlouhak
Copy link
Contributor Author

dlouhak commented Oct 19, 2023

No description provided? Can you please explain how this happened so we can prevent it in the future?

Git blame will answer your questions. 🙃
Thorough code review (✅) and complex snapshot tests (✅).

@literat literat merged commit 5ad7dc9 into main Oct 23, 2023
29 checks passed
@literat literat deleted the bugfix/spinner-classnames branch October 23, 2023 14:03
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.

3 participants