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

perf: update ts client in the performance tests, cleanup println #1041

Merged
merged 3 commits into from May 17, 2024
Merged

perf: update ts client in the performance tests, cleanup println #1041

merged 3 commits into from May 17, 2024

Conversation

yshyn-iohk
Copy link
Member

@yshyn-iohk yshyn-iohk commented May 13, 2024 •
edited

Description:

  • in the scope of ATL-6404
  • cleanup println from the production code
  • update dependency in the performance test

Checklist:

  • My PR follows the contribution guidelines of this project
  • My PR is free of third-party dependencies that don't comply with the Allowlist
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked the PR title to follow the conventional commit specification

@yshyn-iohk
perf: update ts client in the performance tests, cleanup println, A…
030dd8b 
…TL-6404

Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented May 13, 2024 •
edited

Integration Test Results

14 files  ±0  14 suites  ±0   2s ⏱️ ±0s
30 tests ±0  30 ✅ ±0  0 💤 ±0  0 ❌ ±0 
33 runs  ±0  33 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit cfd24e3. ± Comparison against base commit f5f2d75.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented May 13, 2024 •
edited

Unit Test Results

 91 files  ±0   91 suites  ±0   20m 42s ⏱️ +41s
773 tests ±0  765 ✅ ±0  8 💤 ±0  0 ❌ ±0 
780 runs  ±0  772 ✅ ±0  8 💤 ±0  0 ❌ ±0 

Results for commit cfd24e3. ± Comparison against base commit f5f2d75.

♻️ This comment has been updated with latest results.

@coveralls
Copy link

coveralls commented May 13, 2024 •
edited

Coverage Status

coverage: 53.063% (+0.1%) from 52.923%
when pulling cfd24e3 on performance-tests-update
into f5f2d75 on main.

@yshyn-iohk yshyn-iohk enabled auto-merge (squash) May 17, 2024 06:24
@yshyn-iohk yshyn-iohk merged commit 7d5ceba into main May 17, 2024
10 checks passed
@yshyn-iohk yshyn-iohk deleted the performance-tests-update branch May 17, 2024 06:41
hyperledger-bot added a commit that referenced this pull request May 21, 2024
@hyperledger-bot
chore(release): cut Identus Cloud agent 1.33.1 release
214fd3c 
## [1.33.1](cloud-agent-v1.33.0...cloud-agent-v1.33.1) (2024-05-21)

### Bug Fixes

* broken link for the cloud agent packages in readme file ([#1032](#1032)) ([92d17c2](92d17c2))
* expose new key types in rest api ([#1066](#1066)) ([9ce8d3a](9ce8d3a))
* rename the folder to identus for vc-jwt ([#1063](#1063)) ([364a5dc](364a5dc))

### Performance Improvements

* update ts client in the performance tests, cleanup `println` ([#1041](#1041)) ([7d5ceba](7d5ceba))

Signed-off-by: Allain Magyar <allain.magyar@iohk.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@patlo-iog patlo-iog patlo-iog approved these changes

@todorkoleviohk todorkoleviohk Awaiting requested review from todorkoleviohk todorkoleviohk is a code owner

@amagyar-iohk amagyar-iohk Awaiting requested review from amagyar-iohk amagyar-iohk is a code owner

@mineme0110 mineme0110 Awaiting requested review from mineme0110 mineme0110 is a code owner

@bvoiturier bvoiturier Awaiting requested review from bvoiturier bvoiturier is a code owner

Assignees
No one assigned
Labels
cloud-agent mercury
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@yshyn-iohk @coveralls @patlo-iog