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

Update semantics.md #1818

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update semantics.md #1818

wants to merge 1 commit into from

Conversation

marcoa6
Copy link
Collaborator

@marcoa6 marcoa6 commented Mar 29, 2024

Updated semantics with file cache

Description

Link to the issue in case of a bug fix.

NA

Testing details

  1. Manual - NA
  2. Unit tests - NA
  3. Integration tests - NA

Updated semantics with file cache
@marcoa6 marcoa6 requested a review from a team as a code owner March 29, 2024 22:50
docs/semantics.md Show resolved Hide resolved
docs/semantics.md Show resolved Hide resolved
docs/semantics.md Show resolved Hide resolved
docs/semantics.md Show resolved Hide resolved
@gargnitingoogle
Copy link
Collaborator

@marcoa6 most of the Caching section in the semantics.md doc is now a duplicate of what's there in cloud-doc and linked pages, which leads to redundancy and double to effort to maintain. Should we just have the Cache section point to the cloud-doc page ?

@gargnitingoogle
Copy link
Collaborator

@marcoa6 most of the Caching section in the semantics.md doc is now a duplicate of what's there in cloud-doc and linked pages, which leads to redundancy and double to effort to maintain. Should we just have the Cache section point to the cloud-doc page ?

Discussed this separately with @marcoa6 . We'll do a complete de-dup of semantics.md with the cloud-docs as a separate activity.

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.

None yet

3 participants