[Github-comments] Re: [geany/geany] Add CodeQL Analysis workflow (PR #3769)

2024-03-17 Thread Enrico Tröger via Github-comments
Merged #3769 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3769#event-12143071778 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Add CodeQL Analysis workflow (PR #3769)

2024-03-17 Thread Enrico Tröger via Github-comments
> > > Adding ` workflow_dispatch:` might be a good idea to manually re-run the > > > workflow. > > > > > > I would still recommend this simple change. > > And it seems the configuration still has syntax errors: > > https://github.com/geany/geany/actions/runs/8135027731/workflow > > That seems

[Github-comments] Re: [geany/geany-plugins] Make updatechecker plugin for MacOS (Issue #1314)

2024-03-10 Thread Enrico Tröger via Github-comments
> The update checker for Linux works incorrectly, it checks the Geany > repository for a new version, not the repositories for the current distro. > Most distros don't update versions of software like Geany on LTS versions for > example. So it just feeds frustration for users of such I'd say i

[Github-comments] Re: [geany/geany] Add CodeQL Analysis workflow (PR #3769)

2024-03-10 Thread Enrico Tröger via Github-comments
> Adding ` workflow_dispatch:` might be a good idea to manually re-run the > workflow. I would still recommend this simple change. And it seems the configuration still has syntax errors: https://github.com/geany/geany/actions/runs/8135027731/workflow -- Reply to this email directly or view it

[Github-comments] Re: [geany/geany] Add CodeQL Analysis workflow (PR #3769)

2024-03-10 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -8,6 +8,13 @@ on: push: branches: - master +- '**' I don't see why, at least for this case it seems a bit beyond the goal. If we have use cases for excluding files from triggering a CI run, we can add them but no excluding only two

[Github-comments] Re: [geany/geany-themes] New DarkSpook color scheme (PR #68)

2024-03-03 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. On screenshots/dark-spook.png: Do you mind to replace the screenshot as described above? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-themes/pull/68#discussion_r1510333217 You are receiving this because you are sub

[Github-comments] Re: [geany/geany-themes] New DarkSpook color scheme (PR #68)

2024-03-03 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > +description=Based on the InkPot theme of the same name. +# incremented automatically, do not change manually +version=1 +author=Vukić Srđan +url=https://hardcoder.xyz +# list of each compatible Geany release version +compat=1.22;1.23;1.23.1;1.24;1.25;1.

[Github-comments] [geany/geany-themes] Metallic Bottle: use a slightly darker background color for selections (PR #71)

2024-03-03 Thread Enrico Tröger via Github-comments
I just made the background color for selections a little darker so that it is different from the foreground color. Closes #70. ![Screenshot](https://github.com/geany/geany-themes/assets/617017/472e0715-c1c1-4166-926e-b2260d160687) You can view, comment on, or merge this pull request online at:

[Github-comments] Re: [geany/geany] Add CodeQL Analysis workflow (PR #3769)

2024-03-03 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -8,6 +8,13 @@ on: push: branches: - master +- '**' And it causes an error: ``` Invalid workflow file: .github/workflows/build.yml#L8 You have an error in your yaml syntax on line 8 ``` -- Reply to this email directly or view it on

[Github-comments] Re: [geany/geany] Add CodeQL Analysis workflow (PR #3769)

2024-03-03 Thread Enrico Tröger via Github-comments
Thanks! > Done. Some examples of this workflow include a cron schedule as well. Would > you like that added, and if so, how often should it be scheduled? I would say once in a month is a good start. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3769#

[Github-comments] Re: [geany/geany] Add CodeQL Analysis workflow (PR #3769)

2024-03-03 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -8,6 +8,13 @@ on: push: branches: - master +- '**' Why did you change this, what's the intention? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3769#pullrequestreview-1912989734 You are

[Github-comments] Re: [geany/geany] Add screenshot to README (PR #3760)

2024-03-03 Thread Enrico Tröger via Github-comments
Merged #3760 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3760#event-11990226242 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] 2.0.0: test suite fails with cppcheck 2.13.1 (unrecognized command line option: "--template") (Issue #1304)

2024-03-03 Thread Enrico Tröger via Github-comments
Closed #1304 as completed via #1309. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/1304#event-11990221630 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] Add missing sign for cppcheck --template argument (PR #1309)

2024-03-03 Thread Enrico Tröger via Github-comments
Merged #1309 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1309#event-11990221578 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Python print() doesn't work well. (Issue #3772)

2024-03-03 Thread Enrico Tröger via Github-comments
Closed #3772 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3772#event-11990207405 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Python print() doesn't work well. (Issue #3772)

2024-03-03 Thread Enrico Tröger via Github-comments
On my old Windows 7, a "cmd.exe" is opened and the full output is available and there is a scrollbar to see the first lines of the output. Changing the default execution command to "PowerShell" seems wrong and would require that PowerShell is installed. Maybe you can check if the cmd.exe of newe

[Github-comments] Re: [geany/geany-osx] "Change Font" does not seem to have effect (Issue #53)

2024-03-03 Thread Enrico Tröger via Github-comments
As a side note: > Well, simply that I did not know that 2.0 is available for MacOS already. > Sadly, Geany does not have a builtin update checker. There is the UpdateChecker plugin (https://plugins.geany.org/updatechecker.html) which is probably also included in the macOS bundle. Can this be

[Github-comments] Re: [geany/geany-plugins] Is it possible to open "spelling suggestions" menu using keyboard ? (Issue #1306)

2024-03-03 Thread Enrico Tröger via Github-comments
You can open the editor menu with the "menu" key if your keyboard has such a key and then with by pressing cursor down and cursor right, you are directly in the spelling suggestions menu. Maybe this is already enough? -- Reply to this email directly or view it on GitHub: https://github.com/gea

[Github-comments] Re: [geany/geany] Silent Installation of Geany (Issue #3734)

2024-03-03 Thread Enrico Tröger via Github-comments
Closed #3734 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3734#event-11990154191 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Silent Installation of Geany (Issue #3734)

2024-03-03 Thread Enrico Tröger via Github-comments
I forgot to answer the question from the OP: > Please let me know if there are any parameters to install\select following > features silently. There is no way to select/deselect components with the silent installer. -- Reply to this email directly or view it on GitHub: https://github.com/geany/

[Github-comments] Re: [geany/geany] Add CodeQL Analysis workflow (PR #3769)

2024-02-18 Thread Enrico Tröger via Github-comments
Adding ` workflow_dispatch:` might be a good idea to manually re-run the workflow. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3769#issuecomment-1951381789 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] Add Pinner plugin (PR #1308)

2024-02-18 Thread Enrico Tröger via Github-comments
> @eht16 It looks like we were working on something different but similar? ;) I > think incorporating something like that into this plugin would be suitable. Yes, my approach was for a different use case. I like to have a tab/document always in the visible area of the notebook tab list for easy

[Github-comments] Re: [geany/geany] Add CodeQL Analysis workflow (PR #3769)

2024-02-18 Thread Enrico Tröger via Github-comments
Sounds good. - For the next time, a description to describe what is the PR about and more importantly why, is generally a good idea. - We could add `*.py` to the patterns and `python` to the matrix to also cover the Python helper scripts in the repository. - Also, could you strip the commented c

[Github-comments] Re: [geany/geany] Maximum number of files that can be opened in a single instance of Geany? (Discussion #3754)

2024-02-18 Thread Enrico Tröger via Github-comments
Good guess. I tested this and Windows Explorer in fact spawns a single Geany process per selected file and then this happens what @elextr described above. There is probably no easy solution. Making the initial startup faster until the socket connection is checked or implementing some other kind

[Github-comments] Re: [geany/geany-plugins] Add missing sign for cppcheck --template argument (PR #1309)

2024-02-17 Thread Enrico Tröger via Github-comments
See also #1310 for an improved CI build with using the current "cppcheck" version instead of the dated version from the CI runner image. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1309#issuecomment-1950041403 You are receiving this because y

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 6535163202187cdb3c45068d22261279106e50b3 CI: Download, compile and use latest cppcheck version -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/f897ccf7c22cbe086872e029b0a83ed9c856b7c8..6535163202187cdb3c45068d22261279106e50b3 You are receivi

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. f897ccf7c22cbe086872e029b0a83ed9c856b7c8 Come on -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/f0cec4ecdf1c95a5c54efea00a33cc440319ed4e..f897ccf7c22cbe086872e029b0a83ed9c856b7c8 You are receiving this because you are subscribed to this thre

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. f0cec4ecdf1c95a5c54efea00a33cc440319ed4e Come on -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/27b2128185a9c28a1f86dc8e3e940570c8850889..f0cec4ecdf1c95a5c54efea00a33cc440319ed4e You are receiving this because you are subscribed to this thre

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 27b2128185a9c28a1f86dc8e3e940570c8850889 Come on -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/1fbc03df80e14e36ac17b9dcef508e3bba154e9b..27b2128185a9c28a1f86dc8e3e940570c8850889 You are receiving this because you are subscribed to this thre

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 1fbc03df80e14e36ac17b9dcef508e3bba154e9b Come on -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/42fb8db43fff724bb3f58a28f8c279ae4c284e2a..1fbc03df80e14e36ac17b9dcef508e3bba154e9b You are receiving this because you are subscribed to this thre

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 42fb8db43fff724bb3f58a28f8c279ae4c284e2a Come on -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/aacf5995a4c9cbb8b6020e83eb45f85c4bd6523a..42fb8db43fff724bb3f58a28f8c279ae4c284e2a You are receiving this because you are subscribed to this thre

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. aacf5995a4c9cbb8b6020e83eb45f85c4bd6523a Come on -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/90048a47925b2494de8db6e417fbd907b7baf7fc..aacf5995a4c9cbb8b6020e83eb45f85c4bd6523a You are receiving this because you are subscribed to this thre

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 90048a47925b2494de8db6e417fbd907b7baf7fc Come on -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/66e620321ddaf9093210512a7c1e217ff954fd19..90048a47925b2494de8db6e417fbd907b7baf7fc You are receiving this because you are subscribed to this thre

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 66e620321ddaf9093210512a7c1e217ff954fd19 CI: Download, compile and use latest cppcheck version -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/6bc0f1abe02dae90154588d4e854d48831cc0e87..66e620321ddaf9093210512a7c1e217ff954fd19 You are receivi

[Github-comments] Re: [geany/geany] Maximum number of files that can be opened in a single instance of Geany? (Discussion #3754)

2024-02-17 Thread Enrico Tröger via Github-comments
Bingo. This is the relevant piece of information and without a running instance of Geany, I could instantly reproduce it. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/discussions/3754#discussioncomment-8500329 You are receiving this because you are subscri

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 6bc0f1abe02dae90154588d4e854d48831cc0e87 CI: Download, compile and use latest cppcheck version -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/4315850583156f49d8c20e9803081a3ded92c8aa..6bc0f1abe02dae90154588d4e854d48831cc0e87 You are receivi

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 4315850583156f49d8c20e9803081a3ded92c8aa CI: Download, compile and use latest cppcheck version -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/ca02bfbcb0daba84c456402fc1da7f99c6554525..4315850583156f49d8c20e9803081a3ded92c8aa You are receivi

[Github-comments] Re: [geany/geany] Maximum number of files that can be opened in a single instance of Geany? (Discussion #3754)

2024-02-17 Thread Enrico Tröger via Github-comments
> @eht16 so you think the problem is opening files at the same time, not the > total number open as I thought? I guess it could be, the description is > pretty bad. Yes, I did and yes, the description leaves plenty room for interpretation :(. > In that case, is the problem something to do with

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. ca02bfbcb0daba84c456402fc1da7f99c6554525 CI: Download, compile and use latest cppcheck version -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/35305732195ec19866aa6b2935be80bcab22a47a..ca02bfbcb0daba84c456402fc1da7f99c6554525 You are receivi

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 35305732195ec19866aa6b2935be80bcab22a47a CI: Download, compile and use latest cppcheck version -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/11d9f6edf1560fe04a32b424edd5af9055387cca..35305732195ec19866aa6b2935be80bcab22a47a You are receivi

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 11d9f6edf1560fe04a32b424edd5af9055387cca CI: Download, compile and use latest cppcheck version -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/8e0cf360f7cecaaf28272ea20483e3dc0f8bdba6..11d9f6edf1560fe04a32b424edd5af9055387cca You are receivi

[Github-comments] Re: [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 8e0cf360f7cecaaf28272ea20483e3dc0f8bdba6 CI: Download, compile and use latest cppcheck version -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1310/files/eb8a5d9049ccefba384d355c64e7fe4d207e8bfe..8e0cf360f7cecaaf28272ea20483e3dc0f8bdba6 You are receivi

[Github-comments] [geany/geany-plugins] CI: Download, compile and use latest cppcheck version (PR #1310)

2024-02-17 Thread Enrico Tröger via Github-comments
You can view, comment on, or merge this pull request online at: https://github.com/geany/geany-plugins/pull/1310 -- Commit Summary -- * CI: Download, compile and use latest cppcheck version -- File Changes -- M .github/workflows/build.yml (16) -- Patch Links -- https://github.com/ge

[Github-comments] Re: [geany/geany] Maximum number of files that can be opened in a single instance of Geany? (Discussion #3754)

2024-02-17 Thread Enrico Tröger via Github-comments
Tested on Windows 7 with Geany 2.0 and I can successfully open up to 15 files at once in Windows Explorer by selecting them and use the "Open with Geany" context menu action. I noticed when selecting more than 15 files, the context menu action "Open with Geany" does not appear anymore. This is

[Github-comments] Re: [geany/geany] Japanese(and other) characters show up as boxes in Geany on Windows 10 (#1805)

2024-02-17 Thread Enrico Tröger via Github-comments
I have tested this on Windows 7 with Geany 2.0 and most fonts show the described problem. By just trying to switch the font, I found a few which do work: - Batang - BatangChe - DFKai-SB - Dotum There are probably more fonts which work, I gave up trying at that point. This might be caused by a co

[Github-comments] Re: [geany/geany] Japanese(and other) characters show up as boxes in Geany on Windows 10 (#1805)

2024-02-17 Thread Enrico Tröger via Github-comments
Closed #1805 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1805#event-11836116147 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] Add missing sign for cppcheck --template argument (PR #1309)

2024-02-17 Thread Enrico Tröger via Github-comments
I tested the `--template=gcc` format locally with the latest cppcheck version and it works. The idea by using the specific "gcc" template was probably that the output can be parsed by Geany out of the box using the default error message regex for C/C++ files. But this is just a guess. I notice

[Github-comments] Re: [geany/geany-plugins] Add missing sign for cppcheck --template argument (PR #1309)

2024-02-17 Thread Enrico Tröger via Github-comments
@eht16 pushed 1 commit. 146abc696b2d40ba6178328df4184794e40ce106 Add more custom macro definitions for recent cppcheck versions -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1309/files/fa2975229932c9b4eadbf0c7a249d4d8d541cdbb..146abc696b2d40ba6178328df4184794e40ce106 You ar

[Github-comments] Re: [geany/geany-plugins] Add missing sign for cppcheck --template argument (PR #1309)

2024-02-13 Thread Enrico Tröger via Github-comments
@elextr I couldn't resist to credit this great and very sophisticated change to you as you already solved the mystery in https://github.com/geany/geany-plugins/issues/1304#issuecomment-1912600670! -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/

[Github-comments] [geany/geany-plugins] Add missing sign for cppcheck --template argument (PR #1309)

2024-02-13 Thread Enrico Tröger via Github-comments
Closes #1304. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany-plugins/pull/1309 -- Commit Summary -- * Add missing sign for cppcheck --template argument -- File Changes -- M build/cppcheck.mk (2) -- Patch Links -- https://github.com/gean

[Github-comments] Re: [geany/geany-plugins] Add Pinner plugin (PR #1308)

2024-02-13 Thread Enrico Tröger via Github-comments
> Uhhh, that sounds interesting and like something I want to implement since > some years already :). I just found some old code and it seems I started with a similar approach some time ago: https://gist.github.com/eht16/15afd001f238f77f100a43e4129b8d20 Use what is helpful but with care :smile:.

[Github-comments] Re: [geany/geany-plugins] Add Pinner plugin (PR #1308)

2024-02-13 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > + GeanyDocument *doc = document_get_current(); + + if (doc != NULL && pin_list == NULL) + { + GtkWidget *label = gtk_label_new_with_mnemonic(doc->file_name); + gtk_widget_show(label); + gtk_box_pac

[Github-comments] Re: [geany/geany-plugins] Add Pinner plugin (PR #1308)

2024-02-13 Thread Enrico Tröger via Github-comments
Uhhh, that sounds interesting and like something I want to implement since some years already :). -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1308#issuecomment-1942664495 You are receiving this because you are subscribed to this thread. Mess

[Github-comments] Re: [geany/geany] Bump actions/upload-artifact from 3 to 4 (PR #3765)

2024-02-12 Thread Enrico Tröger via Github-comments
Merged #3765 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3765#event-11780769385 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Bump docker/login-action from 2 to 3 (PR #3764)

2024-02-12 Thread Enrico Tröger via Github-comments
Merged #3764 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3764#event-11780755291 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Bump actions/cache from 3 to 4 (PR #3763)

2024-02-12 Thread Enrico Tröger via Github-comments
Merged #3763 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3763#event-11780742922 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Bump actions/checkout from 3 to 4 (PR #3762)

2024-02-12 Thread Enrico Tröger via Github-comments
Merged #3762 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3762#event-11780721954 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Add dependabot.yml (PR #3758)

2024-02-12 Thread Enrico Tröger via Github-comments
Thanks! -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3758#issuecomment-1939724665 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Add dependabot.yml (PR #3758)

2024-02-12 Thread Enrico Tröger via Github-comments
Merged #3758 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3758#event-11780672301 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Add Dockerfile filedef (PR #3757)

2024-02-12 Thread Enrico Tröger via Github-comments
Full agree with what @elextr said, these were mostly the words already in my mouth, I just didn't speak it explicitly. One addition: after this PR we should *not* have a Docker filetype in Geany and the wiki page as it is now. In the end, we should decide on one solution. I think it might be wo

[Github-comments] Re: [geany/geany] Add dependabot.yml (PR #3758)

2024-02-12 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > @@ -0,0 +1,6 @@ +version: 2 +updates: ```suggestion updates: # Before applying suggested PRs, make sure that the new versions of any updated actions are allowed in https://github.com/organizations/geany/settings/actions. Versions are pinned and rest

[Github-comments] Re: [geany/geany] A picture says more than a 1000 words... (Discussion #3746)

2024-02-10 Thread Enrico Tröger via Github-comments
To the original issue: I think this is a great idea, I find myself moaning on others' project pages when there is no screenshot easily accessable but failed to noticed it here :(. What do you think of https://github.com/eht16/geany/tree/6e2dee3cf3df8d2d446d99c3241fc4bfc5cb8758? It doesn't look

[Github-comments] [geany/geany] Add screenshot to README (PR #3760)

2024-02-10 Thread Enrico Tröger via Github-comments
Related to #3746. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/3760 -- Commit Summary -- * Add screenshot to README -- File Changes -- M README (5) -- Patch Links -- https://github.com/geany/geany/pull/3760.patch https://github.c

[Github-comments] Re: [geany/geany] A picture says more than a 1000 words... (Discussion #3746)

2024-02-10 Thread Enrico Tröger via Github-comments
Ha, https://github.com/topics/geany: we've got fancy new social media preview image. I don't understand how is that related to social media but I don't need to understand this I guess :). The image is displayed on the search results now. THe dimensions are not ideal but good enough for now. Bet

[Github-comments] Re: [geany/geany] MSYS2 package for Geany (Issue #3756)

2024-02-10 Thread Enrico Tröger via Github-comments
JFTR, I commented on https://github.com/msys2/MINGW-packages/pull/19963. Using Meson to build Geany on Windows at this point is very risky and the experiences the people described confirm the early stage of Meson support in Geany on Windows. No surprise. -- Reply to this email directly or view

[Github-comments] Re: [geany/geany] Add Dockerfile filedef (PR #3757)

2024-02-10 Thread Enrico Tröger via Github-comments
See also the already filetype configuration for Docker and Docker-Compose files in the wiki: https://wiki.geany.org/config/docker-compose. It might be the version in the wiki could be updated for recent Docker features but it is already working. -- Reply to this email directly or view it on Gi

[Github-comments] Re: [geany/geany] Add dependabot.yml (PR #3758)

2024-02-10 Thread Enrico Tröger via Github-comments
Yeah, why not. Two remarks: - why Docker? - Automatic updating of the action versions won't work directly as we pinned the actions and their versions in the organization settings. I think this is useful to reduce the risk of getting unwanted actions or code executed in CI. So even after merging

[Github-comments] Re: [geany/geany-themes] New DarkSpook color scheme (PR #68)

2024-01-02 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > +keyword_4=#7F7F7F;;false;true + +identifier=#3CCA62;;false;true +identifier_1=#507875;;false;true +identifier_2=#457C54;;false;true +identifier_3=#256536;;false;true +identifier_4=#8BBC98;;false;true + +string=#ffcd8b;;false;false +string_1=#BFAC91;#1C1F

[Github-comments] Re: [geany/geany] Silent Installation of Geany (Issue #3734)

2024-01-02 Thread Enrico Tröger via Github-comments
Use "/S" to perform a silent installation. Additionally you can use "/D=c:\..." to specify the installation directory. For more details see https://nsis.sourceforge.io/Docs/Chapter3.html#installerusagecommon. I'm not sure but it might be that the silent installation will automatically remove a

[Github-comments] Re: [geany/geany] Space bar and tilde (Issue #3521)

2024-01-02 Thread Enrico Tröger via Github-comments
See https://www.geany.org/documentation/faq/#how-to-change-the-gtk-theme for instructions on how to change the theme. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3521#issuecomment-1873945941 You are receiving this because you are subscribed to this

[Github-comments] Re: [geany/geany-themes] New DarkSpook color scheme (PR #68)

2023-12-23 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > +keyword_4=#7F7F7F;;false;true + +identifier=#3CCA62;;false;true +identifier_1=#507875;;false;true +identifier_2=#457C54;;false;true +identifier_3=#256536;;false;true +identifier_4=#8BBC98;;false;true + +string=#ffcd8b;;false;false +string_1=#BFAC91;#1C1F

[Github-comments] Re: [geany/geany-themes] New DarkSpook color scheme (PR #68)

2023-12-23 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. On screenshots/dark-spook.png: It would be great if you could make a screenshot like described in https://github.com/geany/geany-themes/blob/master/ADDING-A-THEME.md#adding-a-screenshot. -- Reply to this email directly or view it on GitHub: https://gith

[Github-comments] Re: [geany/geany] Geany 4.0, the GTK4 edition (and more) (Discussion #3675)

2023-12-20 Thread Enrico Tröger via Github-comments
I hope GTK3 will be still supported for a long time. GTK4 is more than a revolution IMO. It feels more like another toolkit which is using glib by accident as well. >From a user point of view, here is my personal shitlist for GTK4: - no icons on menu items - no classic file menu - no window posit

[Github-comments] Re: [geany/geany] Cannot open uncompressed PDF file (Issue #3677)

2023-12-20 Thread Enrico Tröger via Github-comments
I propose to close this, it's highly unlikely Geany will ever become a PDF editor. Or is there any reason in keeping this open? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3677#issuecomment-1864422905 You are receiving this because you are subscrib

[Github-comments] Re: [geany/geany-themes] DarkSpook theme (Issue #67)

2023-12-20 Thread Enrico Tröger via Github-comments
The easiest way to add the new theme is to create a pull request (after forking this repository) and add your theme then. Please also see https://github.com/geany/geany-themes/blob/master/ADDING-A-THEME.md before (you might skip the "update metadata" section). -- Reply to this email directly

[Github-comments] Re: [geany/geany] Remove deprecated Doxygen settings (PR #3719)

2023-12-20 Thread Enrico Tröger via Github-comments
I just generated the docs without the changes, comitted the results (`reference` and `xml` directories) into a new GIT repository and did the same with the changes applied. The resulting diff is only about the generated timestamps and the following: ```diff diff --git a/xml/Doxyfile.xml b/xml/D

[Github-comments] Re: [geany/geany] Remove deprecated Doxygen settings (PR #3719)

2023-12-13 Thread Enrico Tröger via Github-comments
> > `doxygen -u` doesn't set it to "TEXT" here but to "YES", not sure why. > > Does it? Here it does set it to `TEXT` as it warns about, Doxygen 1.9.4. 1.9.4 here as well. With current master it does change it to `TEXT`, with the changes of this PR it will stay at `YES`. > Is the complaint new

[Github-comments] Re: [geany/geany] Remove deprecated Doxygen settings (PR #3719)

2023-12-12 Thread Enrico Tröger via Github-comments
`doxygen -u` seems to update a lot more of the config, mostly updating comments. I did not check in detail. We could update the config using the command at all, so far I only edited it manually to get the warnings resolved. The warning only said it is changed because `CLASS_DIAGRAM` were set bef

[Github-comments] [geany/geany] Remove deprecated Doxygen settings (PR #3719)

2023-12-12 Thread Enrico Tröger via Github-comments
This removes the followings warnings when generating the docs: ``` warning: Tag 'LATEX_SOURCE_CODE' at line 1758 of file 'Doxyfile.tmp' has become obsolete. To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u" warning: Tag 'CLASS_DI

[Github-comments] Re: [geany/geany] Portable mode without passing a cmdline param (Discussion #3672)

2023-11-19 Thread Enrico Tröger via Github-comments
> > Actually on Windows, Geany is binary relocatable. > > @eht16 Is windows build with a relative PREFIX? No, it's even dirtier like in https://github.com/geany/geany/blob/88a0bfcb3618b2533a5e7e253c7ef7af3c0b0590/src/utils.c#L2282C1-L2282C2. I'm not proud of this, on Windows we use a different f

[Github-comments] Re: [geany/geany] Portable mode without passing a cmdline param (Discussion #3672)

2023-11-19 Thread Enrico Tröger via Github-comments
> > a little batch file in the directory to execute "bin/geany.exe > > --config=portable_config" or something like this. > > the idea was to avoid using a launcher to have the editor in portable mode, i > know currently this is the only option. Why? Using a little batch script is easy and can b

[Github-comments] Re: [geany/geany] How to force Geany to open unknown files as plain text/ASCII? (Discussion #3687)

2023-11-18 Thread Enrico Tröger via Github-comments
One more addition: Geany could be changed to be able to handle any files (including NULL bytes, without a recognised encoding) but it would require a lot of efforts and it needs at someone who is willing and able to do this and also resources for for reviewing and testing. As said earlier, it i

[Github-comments] Re: [geany/geany] Portable mode without passing a cmdline param (Discussion #3672)

2023-11-18 Thread Enrico Tröger via Github-comments
Actually on Windows, Geany is binary relocatable. So what the OP requests should be possible in theory. But I don't think it's necessary to add anything to Geany itself. @eadmaster I suggest to test just putting the Geany installation directory to an USB drive or whatever medium you prefer and a

[Github-comments] Re: [geany/geany-plugins] Error: Most Plugins not binary compatible with this release of geany. Therefore not available in Plugin Manager Dialog (Issue #1292)

2023-11-18 Thread Enrico Tröger via Github-comments
On Windows, having multiple installations of Geany is not really supported. If I remember correctly, the Geany installer writes the installation directory (and so the path to the plugins location) into the registry. The Geany-Plugins installer reads this registry key to set the installation dire

[Github-comments] Re: [geany/geany] Most recent directory is not remembered (Discussion #3664)

2023-11-01 Thread Enrico Tröger via Github-comments
Is there any point in putting efforts into this? As @elextr pointed out already a good option for the OP problem (`Edit->Preferences->General->Startup->Startup Path`), I think Geany is already customisable enough to leave it as is. -- Reply to this email directly or view it on GitHub: https://g

[Github-comments] Re: [geany/geany] Move "Virtual Spaces" preference to "Editor -> Features" (PR #3643)

2023-11-01 Thread Enrico Tröger via Github-comments
@eht16 approved this pull request. LGTM -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3643#pullrequestreview-1709036794 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Move "Virtual Spaces" preference to "Editor -> Features" (PR #3643)

2023-11-01 Thread Enrico Tröger via Github-comments
@eht16 commented on this pull request. > +True +False +12 + + +True +False +

[Github-comments] Re: [geany/geany] Most recent directory is not remembered (Discussion #3664)

2023-10-30 Thread Enrico Tröger via Github-comments
Just to clarify, the described behavior is not implemented. To me it first read like a bug report but it is not, the current behavior is rather intended. Though it could be changed if someone wants to. It is completely unrelated to Windows, on Linux Geany behaves the same (except that the default

[Github-comments] Re: [geany/geany] Pasting via middle-click clears X selection (Issue #3658)

2023-10-28 Thread Enrico Tröger via Github-comments
So in the end, it should be reported to Scintilla, I think. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3658#issuecomment-1783787337 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Pasting via middle-click clears X selection (Issue #3658)

2023-10-28 Thread Enrico Tröger via Github-comments
I can confirm the behavior @b4n described and this is the result of https://sourceforge.net/p/scintilla/bugs/2287/ which is the result of #2629. The previous behavior (pasting the same selection multiple times) was more convenient, I agree. As Neil stated on https://sourceforge.net/p/scintilla/

[Github-comments] Re: [geany/geany] Replace utils_make_human_readable_str() with g_format_size() (PR #3657)

2023-10-28 Thread Enrico Tröger via Github-comments
Forgot to mention, tested and works perfect. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3657#issuecomment-1783785517 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Replace utils_make_human_readable_str() with g_format_size() (PR #3657)

2023-10-28 Thread Enrico Tröger via Github-comments
Yay! I just noticed, Thunar seems to IEC units and so displays the same values as the old code. Though the GLib docs suggest to rather use the SI units. I don't mind much which unit we use, I'm happy that old, copied, not well understood code will be removed! :tada: -- Reply to this email dir

[Github-comments] Re: [geany/geany] Maybe the default sidebar width is too narrow (Issue #3630)

2023-10-28 Thread Enrico Tröger via Github-comments
As said in #3637, the default sidebar width is increased so that it should match better the default window size. Further improvements like modifying the entry's minimal size or ellipsizing the tab labels can be done separately if someone considers it important enough. -- Reply to this email dir

[Github-comments] Re: [geany/geany] Increase the default width for the sidebar (PR #3637)

2023-10-28 Thread Enrico Tröger via Github-comments
I merged it because this PR was only about to increase the default sidebar width for only the default window size. As most users probably will resize and use a bigger window size after all, I'm not sure if the efforts are worth to also modify the entry's minimal size. -- Reply to this email dir

[Github-comments] Re: [geany/geany] Maybe the default sidebar width is too narrow (Issue #3630)

2023-10-28 Thread Enrico Tröger via Github-comments
Closed #3630 as completed via #3637. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3630#event-10797660596 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Increase the default width for the sidebar (PR #3637)

2023-10-28 Thread Enrico Tröger via Github-comments
Merged #3637 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3637#event-10797660582 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Convert JSON to builtin filetype and use Scintilla JSON lexer for it (PR #3647)

2023-10-23 Thread Enrico Tröger via Github-comments
Do we have to convert the existing filetype to a builtin filetype? I guess the answer is yes because the `lexer_filetype` setting can only reference existing filetypes in Geany? I don't want to say we should not do this, but maybe we should remind us ourselves by directly adding a NEWS item for

[Github-comments] Re: [geany/geany] Feature request: Split Norwegian into `nb_NO` and `nn_NO` (Issue #3638)

2023-10-23 Thread Enrico Tröger via Github-comments
> > Ubuntu used one of those and just had to remove its ISOs because a > > translation from such a source contained inappropriate material. At least > > the current system requires a pull request which puts the submitter's > > GitHub id on it so its not totally anonymous. > > That's not a valid

[Github-comments] Re: [geany/geany] Add meson.build to release howto and replace IRC by Matrix (PR #3633)

2023-10-23 Thread Enrico Tröger via Github-comments
Merged #3633 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3633#event-10743770051 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Menu item string "Without encoding (None)" is not fully translatable (Issue #3624)

2023-10-22 Thread Enrico Tröger via Github-comments
I agree with removing "(None)" from the string as it is redundant and making it translatable does not give any value. See #3640. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3624#issuecomment-1774082839 You are receiving this because you are subscri

<    1   2   3   4   5   6   7   8   9   10   >