From d32ddfba1aceac782eb802bbe66f7f1caf7bb0ef Mon Sep 17 00:00:00 2001 From: Hatice Yildirim <Hatice.Yildirim@student.reutlingen-university.de> Date: Sun, 13 Apr 2025 10:36:49 +0200 Subject: [PATCH 1/4] Add ChatGPT scoring results for 5 commits from the Excel sheet 'Good' --- README.md | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index 68219a8..b583ecf 100644 --- a/README.md +++ b/README.md @@ -10,4 +10,13 @@ Du bist ein Experte für Softwareentwicklung und Git. Deine Aufgabe ist es, Git- 6. Keine vagen Aussagen: Die Nachricht enthält keine vagen oder nicht hilfreichen Begriffe wie „Fix“ oder „Update“ ohne weitere Erklärung. 7. Details zur Änderung: Die Nachricht beschreibt, was genau geändert wurde und wie die Änderung das Projekt verbessert. -Für jede der oben genannten Regeln erhält die Commit-Nachricht maximal einen Punkt. Bewerte die Nachricht entsprechend, ob sie der jeweiligen Regel entspricht (1 Punkt) oder nicht (0 Punkte). Das maximale Bewertungsergebnis sind 7 Punkte. \ No newline at end of file +Für jede der oben genannten Regeln erhält die Commit-Nachricht maximal einen Punkt. Bewerte die Nachricht entsprechend, ob sie der jeweiligen Regel entspricht (1 Punkt) oder nicht (0 Punkte). Das maximale Bewertungsergebnis sind 7 Punkte. + +## Bewertung von 5 Git Commit Messages (Excel-Tabelle "Good") +| Commit Message | Score by ChatGPT | Score by Copilot | +|-----------------|-------------------|-------------------| +| Fix crash on login due to null user token | 4 | | +| Rework the @PropertySource parsing logic recently changed in commit 7c608886 to deal with the same source appearing on a @Configuration class and an @Import class | 4 | | +| Issue: SPR-12059 | 1 | | +| Update HierarchicalUriComponents.toUri() to only prepend a missing '/' when the scheme, user info, host or port are specified. This makes the toUri() method behave in the same way as .toUriString() and allows relative URIs to be created. | 5 | | +| UriComponentsBuilder supports query without value | 4 | | \ No newline at end of file -- GitLab From 2f2679126b5c525229c1425a2b84c811b44644d3 Mon Sep 17 00:00:00 2001 From: Hatice Yildirim <Hatice.Yildirim@student.reutlingen-university.de> Date: Sun, 13 Apr 2025 10:39:14 +0200 Subject: [PATCH 2/4] Add Copilot scoring results for 5 commits from the Excel sheet 'Good' --- README.md | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/README.md b/README.md index b583ecf..0320672 100644 --- a/README.md +++ b/README.md @@ -15,8 +15,8 @@ Für jede der oben genannten Regeln erhält die Commit-Nachricht maximal einen P ## Bewertung von 5 Git Commit Messages (Excel-Tabelle "Good") | Commit Message | Score by ChatGPT | Score by Copilot | |-----------------|-------------------|-------------------| -| Fix crash on login due to null user token | 4 | | -| Rework the @PropertySource parsing logic recently changed in commit 7c608886 to deal with the same source appearing on a @Configuration class and an @Import class | 4 | | -| Issue: SPR-12059 | 1 | | -| Update HierarchicalUriComponents.toUri() to only prepend a missing '/' when the scheme, user info, host or port are specified. This makes the toUri() method behave in the same way as .toUriString() and allows relative URIs to be created. | 5 | | -| UriComponentsBuilder supports query without value | 4 | | \ No newline at end of file +| Fix crash on login due to null user token | 4 | 4 | +| Rework the @PropertySource parsing logic recently changed in commit 7c608886 to deal with the same source appearing on a @Configuration class and an @Import class | 4 | 4 | +| Issue: SPR-12059 | 1 | 1 | +| Update HierarchicalUriComponents.toUri() to only prepend a missing '/' when the scheme, user info, host or port are specified. This makes the toUri() method behave in the same way as .toUriString() and allows relative URIs to be created. | 5 | 4 | +| UriComponentsBuilder supports query without value | 1 | 1 | \ No newline at end of file -- GitLab From 600597fa4aba29c799e6d25bce71fafeea240299 Mon Sep 17 00:00:00 2001 From: Hatice Yildirim <Hatice.Yildirim@student.reutlingen-university.de> Date: Sun, 13 Apr 2025 10:48:31 +0200 Subject: [PATCH 3/4] Add ChatGPT scoring results for 5 commits from the Excel sheet 'Bad' --- README.md | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index 0320672..8589d6a 100644 --- a/README.md +++ b/README.md @@ -19,4 +19,14 @@ Für jede der oben genannten Regeln erhält die Commit-Nachricht maximal einen P | Rework the @PropertySource parsing logic recently changed in commit 7c608886 to deal with the same source appearing on a @Configuration class and an @Import class | 4 | 4 | | Issue: SPR-12059 | 1 | 1 | | Update HierarchicalUriComponents.toUri() to only prepend a missing '/' when the scheme, user info, host or port are specified. This makes the toUri() method behave in the same way as .toUriString() and allows relative URIs to be created. | 5 | 4 | -| UriComponentsBuilder supports query without value | 1 | 1 | \ No newline at end of file +| UriComponentsBuilder supports query without value | 1 | 1 | + +## Bewertung von 5 Git Commit Messages (Excel-Tabelle "Bad") +| Commit Message | Score by ChatGPT | Score by Copilot | +|-----------------|-------------------|-------------------| +| refactoring .testsuite .aop.aspectj tests in preparation for migration to .context | 3 | | +| re-adding mock.jndi.* artifacts to .orm test codebase eliminated after moving away from svn:externals | 4 | | +| IN PROGRESS - issue SPR-5599: Hook @Configuration class processing into <context:component-scan/> and <context:annotation-config/> through AnnotationConfigUtils#registerAnnotationConfigProcessors | 3 | | +| + Adding POC tooling integration points, namely AbstractConfigurationPostProcessor and allowing for tooling-specified ClassLoader for use with ASM parsing ++ Eliminated ModelMethod in favor of BeanMethod throughout | 6 | | +| polishing | 2 | | \ No newline at end of file -- GitLab From 53ed2dee04d1364af059e82d5c9539fc0d487838 Mon Sep 17 00:00:00 2001 From: Hatice Yildirim <Hatice.Yildirim@student.reutlingen-university.de> Date: Sun, 13 Apr 2025 11:21:15 +0200 Subject: [PATCH 4/4] Add Copilot scoring results for 5 commits from the Excel sheet 'Bad' --- README.md | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/README.md b/README.md index 8589d6a..1163701 100644 --- a/README.md +++ b/README.md @@ -24,9 +24,8 @@ Für jede der oben genannten Regeln erhält die Commit-Nachricht maximal einen P ## Bewertung von 5 Git Commit Messages (Excel-Tabelle "Bad") | Commit Message | Score by ChatGPT | Score by Copilot | |-----------------|-------------------|-------------------| -| refactoring .testsuite .aop.aspectj tests in preparation for migration to .context | 3 | | -| re-adding mock.jndi.* artifacts to .orm test codebase eliminated after moving away from svn:externals | 4 | | -| IN PROGRESS - issue SPR-5599: Hook @Configuration class processing into <context:component-scan/> and <context:annotation-config/> through AnnotationConfigUtils#registerAnnotationConfigProcessors | 3 | | -| + Adding POC tooling integration points, namely AbstractConfigurationPostProcessor and allowing for tooling-specified ClassLoader for use with ASM parsing -+ Eliminated ModelMethod in favor of BeanMethod throughout | 6 | | -| polishing | 2 | | \ No newline at end of file +| refactoring .testsuite .aop.aspectj tests in preparation for migration to .context | 3 | 3 | +| re-adding mock.jndi.* artifacts to .orm test codebase eliminated after moving away from svn:externals | 4 | 4 | +| IN PROGRESS - issue SPR-5599: Hook @Configuration class processing into <context:component-scan/> and <context:annotation-config/> through AnnotationConfigUtils#registerAnnotationConfigProcessors | 3 | 3 | +| + Adding POC tooling integration points, namely AbstractConfigurationPostProcessor and allowing for tooling-specified ClassLoader for use with ASM parsing + Eliminated ModelMethod in favor of BeanMethod throughout | 6 | 4 | +| polishing | 2 | 2 | \ No newline at end of file -- GitLab