Skip to content

Update HDTestReport #18956

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
Open
jecisc wants to merge 3 commits into
base: Pharo14
Choose a base branch
from
Open

Update HDTestReport #18956

jecisc wants to merge 3 commits into from
+44 −54

Conversation

@jecisc
Copy link
Member

@jecisc jecisc commented Nov 23, 2025 •
edited
Loading

List of changes:

  • Commit an automatic transformation
  • Simplify the logging mechanism (before we had one for the STDout et one for the progress.log file. Now we only have one way to log infos)
  • Simplify the API. The API was confusing because #runAll was a public API but could not be used by itself, it bypassed the #setUp and #tearDown
  • Remove double closing the progress file

Fixes #18954

@jecisc jecisc added the Status: Need more work The issue is nearly ready. Waiting some last bits. label Nov 23, 2025
@jecisc jecisc added Status: Need more work The issue is nearly ready. Waiting some last bits. and removed Status: Need more work The issue is nearly ready. Waiting some last bits. labels Nov 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Status: Need more work The issue is nearly ready. Waiting some last bits.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

(HDTestReport runSuite: PharoDocCommentTest suite) runAll

1 participant

@jecisc