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

Add clippy and fmt checkt to e2e_tests #561

Merged
merged 4 commits into from
Nov 29, 2021

Conversation

ionut-arm
Copy link
Member

Adding extra checks in ci.sh to ensure that the e2e_tests crate is also
correctly formatted and that clippy is fine with its contents.

A number of fixes are also implemented to make clippy happy.

Signed-off-by: Ionut Mihalcea ionut.mihalcea@arm.com

@ionut-arm ionut-arm added code health Issues concerning overall code quality, safety and best practice testing Task related to testing labels Nov 25, 2021
@ionut-arm ionut-arm requested a review from anta5010 November 25, 2021 16:43
@ionut-arm ionut-arm self-assigned this Nov 25, 2021
@ionut-arm ionut-arm requested a review from a team as a code owner November 25, 2021 16:43
@ionut-arm ionut-arm force-pushed the test-cleanup branch 2 times, most recently from e71de70 to 671bde4 Compare November 26, 2021 11:57
Adding extra checks in ci.sh to ensure that the e2e_tests crate is also
correctly formatted and that clippy is fine with its contents.

A number of fixes are also implemented to make clippy happy.

Signed-off-by: Ionut Mihalcea <ionut.mihalcea@arm.com>
Signed-off-by: Ionut Mihalcea <ionut.mihalcea@arm.com>
Signed-off-by: Ionut Mihalcea <ionut.mihalcea@arm.com>
Signed-off-by: Ionut Mihalcea <ionut.mihalcea@arm.com>
@ionut-arm ionut-arm merged commit 0a96a66 into parallaxsecond:main Nov 29, 2021
@ionut-arm ionut-arm deleted the test-cleanup branch November 29, 2021 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code health Issues concerning overall code quality, safety and best practice testing Task related to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants