-
-
Notifications
You must be signed in to change notification settings - Fork 325
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
2021 edition #664
Merged
Merged
2021 edition #664
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: clux <[email protected]>
k, this worked fine after getting a new muslrust:stable pushed :-) |
clux
added a commit
that referenced
this pull request
Oct 22, 2021
* add Resource::object_ref helper to simplify event recording Signed-off-by: clux <[email protected]> * a few more simplifications to events recorder Signed-off-by: clux <[email protected]> * fmt Signed-off-by: clux <[email protected]> * clippy + broken doc links Signed-off-by: clux <[email protected]> * one line clarification Signed-off-by: clux <[email protected]> * remove remnants of derive feature from kube-client Signed-off-by: clux <[email protected]> * only clone what is necessary Signed-off-by: clux <[email protected]> * code review doc comments Signed-off-by: clux <[email protected]> * 2021 edition (#664) Signed-off-by: clux <[email protected]>
clux
added a commit
that referenced
this pull request
Oct 22, 2021
* move kube -> kube-client and try to make a facade crate Signed-off-by: clux <[email protected]> * move relative imports from kube-runtime -> kube-client Signed-off-by: clux <[email protected]> * client: fd -e rs | xargs sed -i 's/kube\:\:/kube_client\:\:/g' also add a standalone readme Signed-off-by: clux <[email protected]> * kube-derive path correcting (now always points at kube-core) Signed-off-by: clux <[email protected]> * kube-derive to point dev-deps at kube-core as well Signed-off-by: clux <[email protected]> * fix kube not re-exporting config when client feature set Signed-off-by: clux <[email protected]> * Update kube/src/lib.rs Co-authored-by: kazk <[email protected]> Signed-off-by: clux <[email protected]> * Update kube-client/README.md Co-authored-by: kazk <[email protected]> Signed-off-by: clux <[email protected]> * ensure kube::core == kube-client::core == kube-core so that we can use same paths equally depending on where we require them Signed-off-by: clux <[email protected]> * add #[kube(kube_crate = "kube")] attr defaults to kube and will append ::core if kube or kube_client but users can also use it with kube_core directly Signed-off-by: clux <[email protected]> * revert change to kube-dervive dev-dependencies they need to look like they came from the facade crate Signed-off-by: clux <[email protected]> * rename to #[kube(kube...)] and document Signed-off-by: clux <[email protected]> * doc tweaks and fixes Signed-off-by: clux <[email protected]> * start writing an architecture md Signed-off-by: clux <[email protected]> * fix imports from new events moudle after crate-split Signed-off-by: clux <[email protected]> * architecture mostly done Signed-off-by: clux <[email protected]> * kube root crate documentation and fmt Signed-off-by: clux <[email protected]> * maintain kube doc imports from kube-client Signed-off-by: clux <[email protected]> * use kube imports in examples Signed-off-by: clux <[email protected]> * Allow `#[derive(CustomResource)]` to take an arbitrary path to `kube_core` (#658) * Allow `#[derive(CustomResource)]` to take an arbitrary path to `kube_core` Signed-off-by: Teo Klestrup Röijezon <[email protected]> * Update kube-derive/src/lib.rs Co-authored-by: Eirik A <[email protected]> Co-authored-by: Eirik A <[email protected]> Signed-off-by: clux <[email protected]> * Update kube-derive/src/lib.rs Co-authored-by: kazk <[email protected]> Signed-off-by: clux <[email protected]> minor doc improvements Signed-off-by: clux <[email protected]> Update architecture.md Co-authored-by: kazk <[email protected]> Signed-off-by: clux <[email protected]> Update architecture.md Co-authored-by: kazk <[email protected]> Signed-off-by: clux <[email protected]> Update architecture.md Co-authored-by: kazk <[email protected]> Signed-off-by: clux <[email protected]> Update architecture.md Co-authored-by: kazk <[email protected]> Signed-off-by: clux <[email protected]> Update architecture.md Co-authored-by: kazk <[email protected]> Signed-off-by: clux <[email protected]> Update examples/README.md Co-authored-by: kazk <[email protected]> Signed-off-by: clux <[email protected]> Update architecture.md Co-authored-by: kazk <[email protected]> Update architecture.md Co-authored-by: kazk <[email protected]> Update architecture.md Co-authored-by: kazk <[email protected]> Update architecture.md Co-authored-by: kazk <[email protected]> * conditions::is_accepted for crds with await_condition (#659) * add conditions::is_accepted for crds - closes #655 Signed-off-by: clux <[email protected]> * rename to established based on new information + timout guard Signed-off-by: clux <[email protected]> * simplify a bit Signed-off-by: clux <[email protected]> * traitify condition Signed-off-by: clux <[email protected]> * docs for Condition fn trait Signed-off-by: clux <[email protected]> * add example for await_condition as well Signed-off-by: clux <[email protected]> * fix pedantic lints Signed-off-by: clux <[email protected]> * add root docs for condition waiting and fill doc gaps in runtime Signed-off-by: clux <[email protected]> * runtime::events: remove client validation + simplify module (#662) - simpler names for snappier docs - removes client side validation - see #654 - everything in one file in root (200 lines after all) - avoids clippy lint Signed-off-by: clux <[email protected]> * Update architecture.md Co-authored-by: Teo Klestrup Röijezon <[email protected]> Update architecture.md Co-authored-by: Teo Klestrup Röijezon <[email protected]> Update architecture.md Co-authored-by: Teo Klestrup Röijezon <[email protected]> Signed-off-by: clux <[email protected]> * change core example in kube-client root Signed-off-by: clux <[email protected]> * add overlaps and crate delineation at the end Signed-off-by: clux <[email protected]> * Events api improvements (#663) * add Resource::object_ref helper to simplify event recording Signed-off-by: clux <[email protected]> * a few more simplifications to events recorder Signed-off-by: clux <[email protected]> * fmt Signed-off-by: clux <[email protected]> * clippy + broken doc links Signed-off-by: clux <[email protected]> * one line clarification Signed-off-by: clux <[email protected]> * remove remnants of derive feature from kube-client Signed-off-by: clux <[email protected]> * only clone what is necessary Signed-off-by: clux <[email protected]> * code review doc comments Signed-off-by: clux <[email protected]> * 2021 edition (#664) Signed-off-by: clux <[email protected]> Co-authored-by: kazk <[email protected]> Co-authored-by: Teo Klestrup Röijezon <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
cargo fix actually did nothing for me, so just manually removed some of the imports from the new prelude