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

More Refactoring #103

Merged
merged 12 commits into from
Nov 30, 2020
Merged

More Refactoring #103

merged 12 commits into from
Nov 30, 2020

Conversation

rylev
Copy link
Contributor

@rylev rylev commented Nov 30, 2020

This continues the refactoring of the cosmos crate to get it to a 0.1 release.

Highlights of these changes include:

  • More documentation! The aim is to be able to use #[deny(missing_docs)]. We're making good progress towards this goal.
  • ResourceType moves into the resources module.
  • Create a headers module for all headers related functionality.
  • Less implementation details are public (including ResourceType and the headers module).
  • Move options like IndexingDirective and Query out of the top level module and into the appropriate resource modules

@rylev
Copy link
Contributor Author

rylev commented Nov 30, 2020

Here's how the top level docs are looking now 😊

image

For reference here's how it looks in master:

image

@rylev rylev added the Cosmos The azure_cosmos crate label Nov 30, 2020
Copy link
Contributor

@MindFlavor MindFlavor left a comment

Choose a reason for hiding this comment

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

Love it 👍

@rylev rylev merged commit 2b88274 into Azure:master Nov 30, 2020
@rylev rylev deleted the more-arrangement branch November 30, 2020 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cosmos The azure_cosmos crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants