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.

Sign up for GitHub

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

Jump to bottom

Cargo clippy for drasi-platform #64

Merged
merged 28 commits into from
Sep 26, 2024
Merged

Cargo clippy for drasi-platform #64

merged 28 commits into from
Sep 26, 2024

Conversation

ruokun-niu
Copy link
Contributor

@ruokun-niu ruokun-niu commented Sep 20, 2024 •
edited
Loading

Description

  • Added a cargo clippy check for the platform repo that checks all rust packages
  • Added a cargo fmt check
  • Query API and change router are skipped for now as there are open PRs

Type of change

  • This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Drasi (issue link optional).

Fixes: #issue_number

@ruokun-niu
move changes from personal fork to upstream branch (#63)
6f468ae 
* dependabot

* dependabot

* dependabot

* wip

* change-dispatcher cargo clippy fix

* publish-api

* view-svc

* workflow

* debug

* workflow config

* submodule

* submodule

* lock nit
@ruokun-niu
query-host
a26c621
@ruokun-niu
Merge branch 'cargo-clippy' of https://github.com/drasi-project/drasi…
30ae04f 
…-platform into cargo-clippy

* 'cargo-clippy' of https://github.com/drasi-project/drasi-platform:
  move changes from personal fork to upstream branch (#63)
@ruokun-niu
nit
2f36f8c
@ruokun-niu
no-deps
a0d89a6
@ruokun-niu
submodule
81acfae
@ruokun-niu
submodule
ec8ba16
@ruokun-niu
lint.yml
7d00768
@ruokun-niu
k8s provider clippy
c81b9d0
@ruokun-niu
nit
25181b2
@ruokun-niu
completed mgmt_api
7b55f25
@ruokun-niu
workflow
3178548
@ruokun-niu
rust lint
537fc5e
@ruokun-niu
submodule
7ee8831
@ruokun-niu
test
4dfbfaa
@ruokun-niu
protobuf
561fe69
@ruokun-niu
resolve conflict
5a0e631
@ruokun-niu
minor fix
e9d5d49
@ruokun-niu
cargo fmt
bcffdea
@ruokun-niu
submodule for cargo fmt
b455e32
@ruokun-niu ruokun-niu marked this pull request as ready for review September 20, 2024 18:29
@ruokun-niu ruokun-niu merged commit ed338f1 into main Sep 26, 2024
52 checks passed
@danielgerlag danielgerlag deleted the cargo-clippy branch September 26, 2024 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@agentofreality agentofreality agentofreality approved these changes

@danielgerlag danielgerlag danielgerlag approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ruokun-niu @danielgerlag @agentofreality