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

Protected APIs marked with @internal for SharedMap and Counter #7521

Merged
merged 9 commits into from
Sep 28, 2021

Conversation

skylerjokiel
Copy link
Contributor

Adding @internal prevents APIs from being on the public docs.

@skylerjokiel skylerjokiel requested a review from a team as a code owner September 17, 2021 23:58
@github-actions github-actions bot added area: dds Issues related to distributed data structures dependencies Pull requests that update a dependency file public api change Changes to a public API labels Sep 17, 2021
@github-actions github-actions bot requested review from vladsud and anthony-murphy and removed request for a team September 17, 2021 23:58
Copy link
Member

@tylerbutler tylerbutler left a comment

Choose a reason for hiding this comment

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

Address the missing exports and ship it. :)

Comment on lines 173 to 174
// Warning: (ae-forgotten-export) The symbol "LocalValueMaker" needs to be exported by the entry point index.d.ts
//
Copy link
Member

Choose a reason for hiding this comment

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

You need to export this type too.

Comment on lines 192 to 193
// Warning: (ae-forgotten-export) The symbol "IDirectoryOperation" needs to be exported by the entry point index.d.ts
//
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

@sumedhb1995
Copy link
Contributor

sumedhb1995 commented Sep 18, 2021

Build blocker is probably being caused by the package-lock and lerna-package-lock changes. Leaving those out of the PR should allow it to pass

@curtisman
Copy link
Member

"lockfileVersion": 2,

are we forcing npm 7?


Refers to: package-lock.json:4 in ce6320e. [](commit_id = ce6320e, deletion_comment = False)

@github-actions github-actions bot removed the dependencies Pull requests that update a dependency file label Sep 19, 2021
@skylerjokiel skylerjokiel requested a review from a team as a code owner September 28, 2021 18:40
@github-actions github-actions bot removed the request for review from a team September 28, 2021 18:40
@github-actions github-actions bot added area: framework Framework is a tag for issues involving the developer framework. Eg Aqueduct area: website labels Sep 28, 2021
@github-actions
Copy link
Contributor

Broken link report 👍🏻

🔗 No broken links found! ✅

Your attention to detail is admirable.

@scottn12 scottn12 merged commit 4c4e926 into microsoft:main Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: dds Issues related to distributed data structures area: framework Framework is a tag for issues involving the developer framework. Eg Aqueduct area: website public api change Changes to a public API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants