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: window not defined - causing build to fail #19

Merged
merged 1 commit into from
May 18, 2022

Conversation

ramit-mitra
Copy link
Contributor

@netlify
Copy link

netlify bot commented May 18, 2022

Deploy Preview for react-plock canceled.

Name Link
🔨 Latest commit 3d7b6ee
🔍 Latest deploy log https://app.netlify.com/sites/react-plock/deploys/62853925efe71800099b66d8

@askides
Copy link
Owner

askides commented May 18, 2022

Hi @ramit-mitra thanks for the PR.

Can i ask you to amend your commit using this style?

fix: window not defined - causing build to fail.

Because the CI will look at the fix: prefix to create a new release.

Another question, is it possible for you to write a test case for this particular event?

Thanks

@ramit-mitra ramit-mitra changed the title Fixing bug - window not defined - causing build to fail fix: window not defined - causing build to fail May 18, 2022
@ramit-mitra
Copy link
Contributor Author

Hey @itsrennyman,
I have updated the PR title. Sadly, am not really sure how to write a test to cover this occurrence.
Thanks

@askides askides merged commit 5d15681 into askides:main May 18, 2022
@askides
Copy link
Owner

askides commented May 18, 2022

🎉 This PR is included in version 1.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ramit-mitra ramit-mitra deleted the bugfix/window-not-defined branch May 20, 2022 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants