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

feat: support import result notify #510

Merged
merged 6 commits into from
Apr 9, 2024
Merged

Conversation

caoxing9
Copy link
Contributor

No description provided.

@caoxing9 caoxing9 marked this pull request as draft March 28, 2024 07:30
@coveralls
Copy link
Collaborator

coveralls commented Mar 28, 2024

Pull Request Test Coverage Report for Build 8610009077

Details

  • 76 of 173 (43.93%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+62.2%) to 82.599%

Changes Missing Coverage Covered Lines Changed/Added Lines %
apps/nestjs-backend/src/features/import/open-api/import.class.ts 28 31 90.32%
apps/nestjs-backend/src/features/import/open-api/import-open-api.service.ts 38 69 55.07%
apps/nestjs-backend/src/features/notification/notification.service.ts 3 66 4.55%
Totals Coverage Status
Change from base Build 8610000456: 62.2%
Covered Lines: 27042
Relevant Lines: 32739

💛 - Coveralls

@caoxing9 caoxing9 force-pushed the feat/import-result-notify branch from 57da49e to 11c5420 Compare March 29, 2024 04:19
@caoxing9 caoxing9 force-pushed the feat/import-result-notify branch 4 times, most recently from 6ef5e65 to e7ade46 Compare April 1, 2024 08:48
@caoxing9 caoxing9 marked this pull request as ready for review April 2, 2024 09:56
@caoxing9 caoxing9 force-pushed the feat/import-result-notify branch from b596e7c to 52861d9 Compare April 9, 2024 03:23
@caoxing9 caoxing9 force-pushed the feat/import-result-notify branch from 52861d9 to 832c391 Compare April 9, 2024 03:42
@caoxing9 caoxing9 merged commit 0700446 into develop Apr 9, 2024
8 checks passed
@caoxing9 caoxing9 deleted the feat/import-result-notify branch April 9, 2024 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants