Skip to content

feat[output-mapping]: Update to newer Monolog#384

Merged
pepamartinec merged 4 commits into
mainfrom
pepa_PAT-221_updateLogger
Apr 24, 2025
Merged

feat[output-mapping]: Update to newer Monolog#384
pepamartinec merged 4 commits into
mainfrom
pepa_PAT-221_updateLogger

Conversation

@pepamartinec
Copy link
Copy Markdown
Contributor

Update Monolog (obecne loggeru), aby slo pouzit psr/log: ^3.0, ktery uz postupne nektery libs zacinaji vyzadovat (treba novy GCP SDK).

Podobny jako keboola/php-component#95

$expectedMessage = 'Cannot delete rows ' .
'from table "in.c-TableDataModifierTest_testDeleteTableRowsWithUnexistColumn.test1" ' .
'in Storage: Cannot filter by column "UnexistColumn", column does not exist';
'in Storage: Filter validation: Cannot filter by column "UnexistColumn", column does not exist';
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.

Tohle nijak nesouvisi s updateme loggeru. Nejspis se v Conneciton zmenil reportovany error a nejakou dobu ted failovalu obecne testy OM i v main.

Copy link
Copy Markdown
Contributor

@romantmb romantmb left a comment

Choose a reason for hiding this comment

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

👍

$testLogger = new TestLogger();
$slicerDecider = new SlicerDecider($testLogger);
$logger = new Logger('test');
;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ti tu něco upadlo 🙂

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.

OMG 😄

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.

To dela to AI

@pepamartinec pepamartinec merged commit 35d08ad into main Apr 24, 2025
15 checks passed
@pepamartinec pepamartinec deleted the pepa_PAT-221_updateLogger branch April 24, 2025 12:25
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.

2 participants