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

Remove default export #511

Closed
wants to merge 1 commit into from
Closed

Remove default export #511

wants to merge 1 commit into from

Conversation

karlhorky
Copy link

@karlhorky karlhorky commented Jun 7, 2022

Fixes #478

See #478 (comment)

This was missing in #507

Example of upgrading to [email protected] and applying this patch here (with tests): https://github.com/upleveled/ical-move-events/pull/119/files


Thanks for contributing to rrule!

To submit a pull request, please verify that you have done the following:

  • Merged in or rebased on the latest master commit
  • Linked to an existing bug or issue describing the bug or feature you're
    addressing
  • Written one or more tests showing that your change works as advertised

@davidgoli
Copy link
Collaborator

The tests will need to be updated as well

@karlhorky
Copy link
Author

Apparently this has been superseded by #513 , closing

@karlhorky karlhorky closed this Jun 8, 2022
@karlhorky karlhorky deleted the patch-1 branch June 8, 2022 13:39
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.

TypeScript + ESM
2 participants