Skip to content

Conversation

@sunrabbit123
Copy link
Contributor

if we failed test code, we need success rate for report

Signed-off-by: sunrabbit123 <qudwls185@naver.com>
@graphite-app
Copy link

graphite-app bot commented Nov 11, 2025

How to use the Graphite Merge Queue

Add the label 0-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

Signed-off-by: sunrabbit123 <qudwls185@naver.com>
Copy link
Contributor

@camc314 camc314 left a comment

Choose a reason for hiding this comment

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

Can you share an example of what this looks like before/after? I thought the errors are already fairly descriptive

@sunrabbit123
Copy link
Contributor Author

yeah!, but we get test code from eslint test set, so we(or I) need statistics on how well the test code follows them.
Of course, this is only for development.
Im talking about how much progress we have left.

스크린샷 2025-11-12 오전 11 15 53

Nothing changes except the line with the emoji at the end.

@auvred
Copy link
Collaborator

auvred commented Nov 12, 2025

Why not just https://github.com/gotestyourself/gotestsum?

@sunrabbit123
Copy link
Contributor Author

good idea, but IMO it's depdendency
I try not to increase my dependence as much as possible.
This is just a personal disposition.

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