imbajin commented on code in PR #3328: URL: https://github.com/apache/hugegraph/pull/3328#discussion_r4254669621
########## .gitattributes: ########## @@ -15,5 +15,8 @@ install-dist/scripts/ export-ignore hugegraph-commons/hugegraph-dist/ export-ignore docker/ export-ignore +# Documentation images are not needed in source releases. +/docs/images/ export-ignore Review Comment: Current-delta source review: 8/10. The existing storage-lifecycle image reference remains unresolved. This latest commit adds another retained-document consumer: helm/hugegraph/README.md:50 now resolves into .github/images, which git archive excludes. Both relative links work in a checkout but lose their targets in the source release. Please align retained documentation and archive contents, either retaining referenced assets or selecting release-safe references. This is one documentation/archive consistency issue family, not a product-runtime failure. No archive or tests were executed in this review. -- 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]
