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

Ignore multiple start() calls #1004

Merged
merged 1 commit into from
Oct 28, 2022
Merged

Ignore multiple start() calls #1004

merged 1 commit into from
Oct 28, 2022

Conversation

kevinmcconnell
Copy link
Contributor

This makes duplicate Trix.start() calls harmless, so that clients don't need to keep track of whether they have previously called it. Previously calling start() a second time raised an error.

This makes it easier for clients that delay importing Trix until it's needed, and may have Trix.start() calls in multiple places.

This makes duplicate `Trix.start()` calls harmless, so that clients
don't need to keep track of whether they have previously called it.
Previously calling `start()` a second time raised an error.

This makes it easier for clients that delay importing Trix until it's
needed, and may have `Trix.start()` calls in multiple places.
Copy link
Contributor

@afcapel afcapel left a comment

Choose a reason for hiding this comment

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

LGTM

@afcapel afcapel merged commit 8600322 into v2 Oct 28, 2022
@afcapel afcapel deleted the prevent-duplicate-starts branch October 28, 2022 14:08
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.

2 participants