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

Introduce spec implementation tests #361

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Introduce spec implementation tests #361

wants to merge 5 commits into from

Conversation

leordev
Copy link
Member

@leordev leordev commented Jul 31, 2024 •
edited
Loading

(wip draft, don't merge it yet)

Stage 1:

  • Run tests on the children specs implementation languages for new Spec PRs
  • Report it back in this Spec PR as a comment

Stage 2:

  • Scheduled run on the children specs repo, like in a daily basis
  • If failure aggregate the results in an issue
  • Each run is logged in that issue with the failed spec repo version

Stage 3:

  • Improve notification: should it go to Discord/Slack/Email/etc?

@leordev leordev force-pushed the leordev/test-implementations branch 11 times, most recently from 8e0295d to 776ad74 Compare August 5, 2024 06:04
@TBD54566975 TBD54566975 deleted a comment from github-actions bot Aug 5, 2024
@TBD54566975 TBD54566975 deleted a comment from github-actions bot Aug 5, 2024
@leordev leordev force-pushed the leordev/test-implementations branch from 776ad74 to f031335 Compare August 5, 2024 06:19
@leordev leordev force-pushed the leordev/test-implementations branch from f031335 to 37e1cd5 Compare August 5, 2024 06:26
@tbd-ci-cd-robot
Copy link

tbdex Spec Implementation SDKs Test Results

Repository Status
tbdex-js ✅ (success)
tbdex-swift ❌ (cancelled)
tbdex-kt ❌ (cancelled)
tbdex-rs ❌ (cancelled)

This is an automated CI report

@leordev leordev force-pushed the leordev/test-implementations branch 10 times, most recently from e264ad2 to 0b81886 Compare August 14, 2024 00:24
@leordev leordev force-pushed the leordev/test-implementations branch from 0b81886 to 084c73b Compare August 14, 2024 00:29
@leordev leordev force-pushed the leordev/test-implementations branch from 084c73b to 1ad7c5e Compare August 14, 2024 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michaelneale michaelneale Awaiting requested review from michaelneale michaelneale will be requested when the pull request is marked ready for review michaelneale is a code owner

@mistermoe mistermoe Awaiting requested review from mistermoe mistermoe will be requested when the pull request is marked ready for review mistermoe is a code owner

@jiyoontbd jiyoontbd Awaiting requested review from jiyoontbd jiyoontbd will be requested when the pull request is marked ready for review jiyoontbd is a code owner

@KendallWeihe KendallWeihe Awaiting requested review from KendallWeihe KendallWeihe will be requested when the pull request is marked ready for review KendallWeihe is a code owner

@kirahsapong kirahsapong Awaiting requested review from kirahsapong kirahsapong will be requested when the pull request is marked ready for review kirahsapong is a code owner

@diehuxx diehuxx Awaiting requested review from diehuxx diehuxx will be requested when the pull request is marked ready for review diehuxx is a code owner

@decentralgabe decentralgabe Awaiting requested review from decentralgabe decentralgabe will be requested when the pull request is marked ready for review decentralgabe is a code owner

@angiejones angiejones Awaiting requested review from angiejones angiejones will be requested when the pull request is marked ready for review angiejones is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@leordev