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

chore: checking documentation #658

Merged
merged 3 commits into from
Nov 4, 2024
Merged

Conversation

tanderson-ld
Copy link
Contributor

No description provided.

@tanderson-ld tanderson-ld requested a review from a team as a code owner November 4, 2024 16:24
Copy link
Contributor

github-actions bot commented Nov 4, 2024

@launchdarkly/js-sdk-common size report
This is the brotli compressed size of the ESM build.
Size: 18669 bytes
Size limit: 21000

Copy link
Contributor

github-actions bot commented Nov 4, 2024

@launchdarkly/js-client-sdk-common size report
This is the brotli compressed size of the ESM build.
Size: 15292 bytes
Size limit: 20000

Copy link
Contributor

github-actions bot commented Nov 4, 2024

@launchdarkly/js-client-sdk size report
This is the brotli compressed size of the ESM build.
Size: 19445 bytes
Size limit: 21000

@@ -1,5 +1,8 @@
import { LDContextMeta } from './LDContextMeta';

/**
* Common attributes shared by the various derived context types.
Copy link
Member

Choose a reason for hiding this comment

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

I think we could drop derived here. It may imply something and I think the meaning is basically the same without it.

@tanderson-ld tanderson-ld merged commit 215721d into main Nov 4, 2024
21 checks passed
@tanderson-ld tanderson-ld deleted the ta/sdk-739/documentation-pass branch November 4, 2024 17:51
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