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

Automation: Main Next Integrate #11637

Merged

Conversation

github-actions[bot]
Copy link
Contributor

Main-next integrate PR

The aim of this pull request is to sync main and next branch. The expectation from the assignee is as follows:

  • Acknowledge the pull request by adding a comment -- "Actively working on it".
  • Resolve any merge conflicts between this branch and next (and push the resolution to this branch). Merge next into this branch if needed. Do NOT rebase or squash this branch: its history must be preserved.
  • Ensure CI is passing for this PR, fixing any issues. Please don't look into resolving Real service e2e test and Stress test failures as they are non-required CI failures.
  • Once the PR is ready for merge, please add a comment @mentioning the reviewers to get this PR merged in. Thank you!!

For more information about how to resolve merge conflicts and CI failures, visit [this wiki page]
(https://github.com/microsoft/FluidFramework/wiki/Main-next-Automation).

curtisman and others added 4 commits August 23, 2022 09:54
Test telemetry indicates that this test has`"R11s fetch error: {\"code\":\"NotFoundError\", ...` during snapshot fetch at loading, This test is expected to throw on connection instead. Additionally, the test doesn't fail from having a different error from expected.

The root cause is that we didn't pass in the resolved URL, because t9s/r11s/afr assign container id and don't use the "name hint".
Fixed it by pass it into `createContainerUrl`.

Additionally, change the test to be more strict about what kind of error it is expecting, so that it doesn't hide hidden issue in the future.
Fills in `SharedCounter` article for fluidframework.com.

Also...
- cleans up some surrounding documentation and related source-code comments.
- removes some empty articles that were hidden from the website, and had no comments or anything in them
- updates page ordering to reflect current navigation
@msfluid-bot
Copy link
Collaborator

@fluid-example/bundle-size-tests: +17.53 KB
Metric NameBaseline SizeCompare SizeSize Diff
aqueduct.js 392.45 KB 394.76 KB +2.32 KB
connectionState.js 680 Bytes 680 Bytes No change
containerRuntime.js 191.92 KB 197.48 KB +5.55 KB
loader.js 151.12 KB 151.06 KB -57 Bytes
map.js 42.63 KB 47.38 KB +4.75 KB
matrix.js 131.63 KB 134.98 KB +3.35 KB
odspDriver.js 150.23 KB 150.11 KB -127 Bytes
odspPrefetchSnapshot.js 38.39 KB 38.35 KB -41 Bytes
sharedString.js 152.42 KB 154.2 KB +1.77 KB
Total Size 1.25 MB 1.27 MB +17.53 KB

Baseline commit: 632468f

Generated by 🚫 dangerJS against 77b3c86

@sonalivdeshpande sonalivdeshpande merged commit 5276235 into next Aug 24, 2022
@sonalivdeshpande sonalivdeshpande deleted the main-next-77b3c86a81543c81a2ea94ec66cceb7c128519d3 branch August 24, 2022 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants