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

ci(i): Add workflow to detect out of date mocks #2679

Merged
merged 4 commits into from
Jun 4, 2024

Conversation

shahzadlone
Copy link
Member

@shahzadlone shahzadlone commented Jun 4, 2024

Relevant issue(s)

Resolves #1616

Description

How has this been tested?

Specify the platform(s) on which this was tested:

  • WSL2

@shahzadlone shahzadlone added the ci/build This is issue is about the build or CI system, and the administration of it. label Jun 4, 2024
@shahzadlone shahzadlone added this to the DefraDB v0.12 milestone Jun 4, 2024
@shahzadlone shahzadlone requested a review from a team June 4, 2024 05:04
@shahzadlone shahzadlone self-assigned this Jun 4, 2024
Copy link

codecov bot commented Jun 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.13%. Comparing base (7dc07e7) to head (231b5da).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2679      +/-   ##
===========================================
- Coverage    78.15%   78.13%   -0.01%     
===========================================
  Files          303      303              
  Lines        23052    23052              
===========================================
- Hits         18014    18011       -3     
- Misses        3676     3679       +3     
  Partials      1362     1362              
Flag Coverage Δ
all-tests 78.13% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 6 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7dc07e7...231b5da. Read the comment docs.

Copy link
Contributor

@AndrewSisley AndrewSisley left a comment

Choose a reason for hiding this comment

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

LGTM - Thanks Shahzad :)

@@ -158,8 +158,8 @@ deps\:chglog:
deps\:modules:
go mod download

.PHONY: deps\:mock
deps\:mock:
.PHONY: deps\:mocks
Copy link
Contributor

Choose a reason for hiding this comment

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

praise: Thanks - this one still trips me up 😁

Copy link
Collaborator

@fredcarle fredcarle left a comment

Choose a reason for hiding this comment

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

LGTM

@shahzadlone shahzadlone force-pushed the lone/ci/mocks-action branch 2 times, most recently from e870009 to a920c06 Compare June 4, 2024 21:02
@shahzadlone shahzadlone force-pushed the lone/ci/mocks-action branch from a920c06 to 231b5da Compare June 4, 2024 23:18
@shahzadlone shahzadlone merged commit 25fc07d into sourcenetwork:develop Jun 4, 2024
34 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/build This is issue is about the build or CI system, and the administration of it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a mandatory ci workflow to indicate when mocks are out dated
3 participants