Actions
Architecture #2167
closedEnforce the event type value serialize
Status:
Released
Priority:
1 (highest)
Assignee:
Category:
Architecture - Code maintenance
Target version:
Pull Request:
Effort required:
Name check:
Fix check:
Regression:
Description
The current code for eventlog has a String for the eventtype, which can be quite error prone and more difficult to maintain
Creating type based event type would prevent typo as the compiler would watch any typo
Updated by Nicolas CHARLES almost 13 years ago
- Status changed from In progress to Pending technical review
- % Done changed from 0 to 100
Updated by François ARMAND almost 13 years ago
- Status changed from Pending technical review to 10
Technical review seems OK.
Updated by Jonathan CLARKE almost 13 years ago
- Status changed from 10 to Released
Closing these "Architecture" issues without a functional review - this step of the workflow has no sense here, there is no functional impact on code architecture changes (or there shouldn't be anyway!).
I think we should remove this step from the workflow for Architecture issues.
Actions