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: update workflows #1511

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

ci: update workflows #1511

wants to merge 6 commits into from

Conversation

amagyar-iohk
Copy link
Contributor

Description:

  • Delete unused workflows
  • Delete unused dockerfile
  • Updates identus integration repository dispatch
  • Change commit analyser in semantic-release to use conventional commits
    • Changes the way changelog is generated
  • Updates changelog to match conventional commit style

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

@amagyar-iohk amagyar-iohk requested a review from a team as a code owner February 3, 2025 13:30
@github-actions github-actions bot added the ci label Feb 3, 2025
@amagyar-iohk amagyar-iohk marked this pull request as draft February 3, 2025 13:32
@amagyar-iohk amagyar-iohk marked this pull request as ready for review February 3, 2025 13:36
Copy link
Contributor

github-actions bot commented Feb 3, 2025

Integration Test Results

 23 files  ±0   23 suites  ±0   3s ⏱️ ±0s
 56 tests ±0   56 ✅ ±0  0 💤 ±0  0 ❌ ±0 
103 runs  ±0  103 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit e4422a1. ± Comparison against base commit d16da04.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Feb 3, 2025

Unit Test Results

105 files  ±0  105 suites  ±0   19m 46s ⏱️ - 3m 22s
889 tests ±0  881 ✅ +12  8 💤 ±0  0 ❌  - 12 
896 runs  ±0  888 ✅ +12  8 💤 ±0  0 ❌  - 12 

Results for commit 837ae77. ± Comparison against base commit 8147901.

♻️ This comment has been updated with latest results.

@coveralls
Copy link

coveralls commented Feb 3, 2025

Coverage Status

coverage: 48.657%. remained the same
when pulling e4422a1 on ci/release-workflow
into d16da04 on main.

@yshyn-iohk
Copy link
Contributor

@amagyar-iohk, can you test these changes in the branch before committing?
From my experience, when there are a lot of modifications the probability of failure is high as well :)

@github-actions github-actions bot added the build label Feb 4, 2025
Signed-off-by: Allain Magyar <[email protected]>
Signed-off-by: Allain Magyar <[email protected]>
Signed-off-by: Allain Magyar <[email protected]>
Signed-off-by: Allain Magyar <[email protected]>
Signed-off-by: Allain Magyar <[email protected]>
Signed-off-by: Allain Magyar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants