@eht16 commented on this pull request.

Thanks for your contribution!

Here are a few remarks:


In colorschemes/carbonfox.conf:

> @@ -0,0 +1,99 @@
+# Colors selected by DrewG @ justaguylinux(dot)com

Could you add a license notice for this and the other themes?

If possible, it would be "GPLv2 or later" to be compatible with Geany. If it's not possible because you decide different or the original templates doesn't allow this, it's ok.
But you should add a license to each theme.


In colorschemes/carbonfox.conf:

> @@ -0,0 +1,99 @@
+# Colors selected by DrewG @ justaguylinux(dot)com
+# Colors adapted from the Carbonfox theme (Nightfox for Ghostty)
+# Author: DG 

Why not your full name (here and in all the other places)?


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany-themes/pull/83/review/2588360822@github.com>