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

procedural data generation for identifiers #15800

Merged
merged 10 commits into from
Jun 13, 2023

Conversation

daesun-park
Copy link
Contributor

@daesun-park daesun-park commented Jun 1, 2023

Description

This PR refactors functions from contextuallyTyped.ts to take in a context object containing an optional globalFieldKeySymbol, to allow generation of missing globalFields which can be used for things like identifiers.

@daesun-park daesun-park requested a review from a team as a code owner June 1, 2023 17:17
@github-actions github-actions bot added area: dds Issues related to distributed data structures area: dds: tree public api change Changes to a public API base: main PRs targeted against main branch labels Jun 1, 2023
@daesun-park daesun-park requested a review from a team as a code owner June 7, 2023 16:08
@github-actions github-actions bot added the area: examples Changes that focus on our examples label Jun 7, 2023
@msfluid-bot
Copy link
Collaborator

msfluid-bot commented Jun 7, 2023

@fluid-example/bundle-size-tests: +413 Bytes
Metric NameBaseline SizeCompare SizeSize Diff
aqueduct.js 448.95 KB 448.95 KB +6 Bytes
connectionState.js 680 Bytes 680 Bytes No change
containerRuntime.js 239.59 KB 239.59 KB +2 Bytes
loader.js 154.65 KB 154.66 KB +4 Bytes
map.js 46.75 KB 46.75 KB +2 Bytes
matrix.js 146.68 KB 146.68 KB +2 Bytes
odspDriver.js 92.46 KB 92.47 KB +6 Bytes
odspPrefetchSnapshot.js 43.69 KB 43.69 KB +4 Bytes
sharedString.js 163.35 KB 163.35 KB +2 Bytes
sharedTree2.js 280.69 KB 281.06 KB +383 Bytes
Total Size 1.74 MB 1.74 MB +413 Bytes

Baseline commit: dc61930

Generated by 🚫 dangerJS against 920c6b1

@daesun-park daesun-park merged commit a51cd2a into microsoft:main Jun 13, 2023
connorskees pushed a commit to connorskees/FluidFramework that referenced this pull request Jul 19, 2023
## Description

This PR refactors functions from `contextuallyTyped.ts` to take in a
`context` object containing a `FieldSource` to generate a `FieldGenerator` used to populate missing (but required) fields based on the schema
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: dds: tree area: dds Issues related to distributed data structures area: examples Changes that focus on our examples base: main PRs targeted against main branch public api change Changes to a public API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants