chibenwa commented on code in PR #2894: URL: https://github.com/apache/james-project/pull/2894#discussion_r2659912678
########## src/adr/0076-deleted-message-vault-single-bucket.md: ########## @@ -0,0 +1,40 @@ +# 76. Deleted Message Vault: single bucket usage Review Comment: +1 on the overall goal to keep James understandable. On my side a conflicting goal come to mind: discoverability. Having a single folder for ADRs makes it a single place to speak architecture and at least I know where to look. There might exist nice solution to conciliate both goals (like referencing new ADR locations in James main ADR location like establishing `mailbox/src/adr` ?) - maybe other projects using ADRs have already soved this problem. Also would we intend to relocate the concerned ADRs? IMO it would be a awesome mailing list topic! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
