Skip to content

docs: start architecture document#1449

Merged
coryan merged 3 commits intogoogleapis:mainfrom
coryan:docs-start-architecture-document
Mar 10, 2025
Merged

docs: start architecture document#1449
coryan merged 3 commits intogoogleapis:mainfrom
coryan:docs-start-architecture-document

Conversation

@coryan
Copy link
Collaborator

@coryan coryan commented Mar 10, 2025

This is where we will document the basic structure of the code, and the
internals of the main libraries.

This is where we will document the basic structure of the code, and the
internals of the main libraries.
@coryan coryan marked this pull request as ready for review March 10, 2025 16:16
@coryan coryan requested review from codyoss and dbolduc March 10, 2025 16:16
@codecov
Copy link

codecov bot commented Mar 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.74%. Comparing base (49dc03e) to head (ea2d4c5).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1449   +/-   ##
=======================================
  Coverage   66.74%   66.74%           
=======================================
  Files          39       39           
  Lines        2195     2195           
=======================================
  Hits         1465     1465           
  Misses        730      730           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

ARCHITECTURE.md Outdated
@@ -0,0 +1,81 @@
# Architectuire Guide

This document describes the high-level architecture of the Google Cloud C++
Copy link
Member

Choose a reason for hiding this comment

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

s/C++/Rust/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Now you know where I cut&pasted this from. Fixed.

ARCHITECTURE.md Outdated
The goal of the libraries is to provide idiomatic Rust libraries to access
services in [Google Cloud](https://cloud.google.com). All services are in scope.
As of 2025-03 we have over 100 libraries, covering most Google Cloud services.
The APIs are not stable
Copy link
Member

Choose a reason for hiding this comment

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

s/stable$/stable.$/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed.

ARCHITECTURE.md Outdated
client.
- `src/lro`: support code for long-running operations.
- `generator/`: the code generator, also known as `sidekick`.
- `src/integration-tests`: the integration tests. This run against production
Copy link
Member

Choose a reason for hiding this comment

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

s/This/These/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed.

README.md Outdated

## Getting Started

The [User Guide] includes basic tutorials to get
Copy link
Member

Choose a reason for hiding this comment

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

"... to get familiar with the Google Cloud Rust SDK."

?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed, thanks.

@coryan coryan enabled auto-merge (squash) March 10, 2025 17:27
@coryan coryan merged commit 75088e5 into googleapis:main Mar 10, 2025
20 checks passed
@coryan coryan deleted the docs-start-architecture-document branch March 10, 2025 17:52
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