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.

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

Feature/add error history module #447

Merged
merged 12 commits into from
Mar 26, 2024
Merged

Conversation

@andistorm andistorm force-pushed the feature/add-error-history-module branch from 38f9074 to 4651128 Compare November 22, 2023 16:10
@andistorm andistorm force-pushed the feature/add-error-history-module branch 5 times, most recently from 6aec2a2 to c362e1c Compare December 5, 2023 16:23
@andistorm andistorm requested a review from hikinggrass December 5, 2023 17:13
@andistorm
Copy link
Contributor Author

Check required ec-cli before merging

@andistorm andistorm force-pushed the feature/add-error-history-module branch 2 times, most recently from d8800c8 to 5f7e726 Compare December 6, 2023 09:39
@andistorm andistorm marked this pull request as ready for review December 6, 2023 10:53
@andistorm
Copy link
Contributor Author

Do not merge before setting correct dependencies in dependencies.yaml

@andistorm andistorm mentioned this pull request Dec 6, 2023
1 task
@andistorm andistorm force-pushed the feature/add-error-history-module branch from 2ae910e to 028d6c8 Compare December 11, 2023 09:32
@corneliusclaussen corneliusclaussen marked this pull request as draft January 18, 2024 16:32
@corneliusclaussen
Copy link
Contributor

Converting to draft as it is waiting for other PRs and cannot be merged now. Also needs cleanup of history after dependent PRs are merged.

@andistorm andistorm force-pushed the feature/add-error-history-module branch from a966608 to 738bd5d Compare January 22, 2024 23:33
@andistorm andistorm requested review from hikinggrass and removed request for hikinggrass January 23, 2024 00:39
@andistorm andistorm force-pushed the feature/add-error-history-module branch from 6c433d4 to 7067f65 Compare January 23, 2024 00:39
dependencies.yaml Outdated Show resolved Hide resolved
dependencies.yaml Outdated Show resolved Hide resolved
dependencies.yaml Outdated Show resolved Hide resolved
@andistorm andistorm marked this pull request as ready for review January 29, 2024 16:18
@andistorm andistorm requested review from hikinggrass and removed request for hikinggrass January 29, 2024 16:44
@hikinggrass hikinggrass force-pushed the feature/add-error-history-module branch 3 times, most recently from 6f85a51 to d00becf Compare March 5, 2024 19:19
@hikinggrass hikinggrass force-pushed the feature/add-error-history-module branch from d00becf to 6603980 Compare March 5, 2024 19:32
@corneliusclaussen corneliusclaussen added the post-release Tag that this PR should not go into the current merge window for the upcoming release label Mar 22, 2024
andistorm and others added 9 commits March 26, 2024 10:38
* Add types file error_history.yaml
* Add interface file error_history.yaml
* Add ErrorDatabaseSQLite as subclass of error/ErrorDatabase
* Implement module
* Add integration tests: error history module
* Add unit tests ErrorDatabaseSQlite

Signed-off-by: Andreas Heinrich <[email protected]>
* use branches in dependencies.yaml
* use build-kit-alpine tag feature/refactor-error-classes

Signed-off-by: Andreas Heinrich <[email protected]>
Signed-off-by: Andreas Heinrich <[email protected]>
Signed-off-by: Andreas Heinrich <[email protected]>
Signed-off-by: Andreas Heinrich <[email protected]>
Signed-off-by: Andreas Heinrich <[email protected]>
These are tested with different tests

Signed-off-by: Kai-Uwe Hermann <[email protected]>
@hikinggrass hikinggrass force-pushed the feature/add-error-history-module branch from 2f4554c to e63bd04 Compare March 26, 2024 09:38
modules/ErrorHistory/tests/CMakeLists.txt Outdated Show resolved Hide resolved
modules/ErrorHistory/tests/CMakeLists.txt Outdated Show resolved Hide resolved
modules/ErrorHistory/tests/CMakeLists.txt Outdated Show resolved Hide resolved
modules/ErrorHistory/tests/CMakeLists.txt Outdated Show resolved Hide resolved
modules/ErrorHistory/CMakeLists.txt Outdated Show resolved Hide resolved
Signed-off-by: Andreas Heinrich <[email protected]>
Signed-off-by: Andreas Heinrich <[email protected]>
@andistorm andistorm merged commit d90d925 into main Mar 26, 2024
4 of 5 checks passed
@andistorm andistorm deleted the feature/add-error-history-module branch March 26, 2024 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
post-release Tag that this PR should not go into the current merge window for the upcoming release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants