Skip to content

projectui tests are missing from CI checks - #9603

Open
jtulach wants to merge 4 commits into
apache:masterfrom
jtulach:jtulach/EnableProjectUiTestsInCi
Open

projectui tests are missing from CI checks#9603
jtulach wants to merge 4 commits into
apache:masterfrom
jtulach:jtulach/EnableProjectUiTestsInCi

Conversation

@jtulach

@jtulach jtulach commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

@jtulach
jtulach requested a review from mbien September 5, 2026 02:39
@jtulach jtulach self-assigned this Sep 5, 2026
boolean ok;

@Override
public void publish(LogRecord record) {

@jtulach jtulach Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

  • Logging handler to check whether something is happening deep in the code
    • and it is not happening
  • I guess we can just delete the test...
    • it must be some hardcore regression test
    • when it is using the log Handler in such an advanced way
    • maybe I wrote, it but I don't remember what it was supposed to do
  • anyway we lived without that tests for some time, we can live without it in the future

@jtulach
jtulach requested a review from lahodaj September 5, 2026 05:29
@mbien mbien added tests CI continuous integration changes labels Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI continuous integration changes tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants