Skip to content

Conversation

@kinyoklion
Copy link
Member

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

Provide links to any issues in this repository or elsewhere relating to this pull request.

Describe the solution you've provided

Provide a clear and concise description of what you expect to happen.

Describe alternatives you've considered

Provide a clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context about the pull request here.

*
* Generally this event should not be created directly and instead an
* {@link MigrationOpTracker} should be used to generate it.
* LDMigrationOpTracker should be used to generate it.
Copy link
Member Author

Choose a reason for hiding this comment

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

Removing this link, because it doesn't import it.

export * from './data';
export * from './options';
export * from './LDClient';
export * from './LDMigration';
Copy link
Member Author

Choose a reason for hiding this comment

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

This was missed.

@kinyoklion kinyoklion merged commit 1aaaec7 into main Oct 16, 2023
@kinyoklion kinyoklion deleted the rlamb/export-migration-apis branch October 16, 2023 22:45
@github-actions github-actions bot mentioned this pull request Oct 16, 2023
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.

3 participants