Skip to content

Add OpenTrader - #10413

Merged
probonopd merged 1 commit into
masterfrom
discover/OpenTrader
Oct 18, 2026
Merged

probonopd merged 1 commit into
masterfrom
discover/OpenTrader

Conversation

@probonopd

Copy link
Copy Markdown
Member

Repository: https://github.com/deepentropy/opentrader

Free, open-source desktop charting and trading app for Windows, macOS and Linux.

  • Stars: 11
  • Release with the AppImage: v0.4.3 (2026-10-18T08:51:08Z)
  • AppImage asset: OpenTrader_linux_amd64.AppImage
  • License: MIT

Found by .github/workflows/discover-apps.yml, which looks for GitHub
repositories that publish AppImages but are not in the catalog yet. This
entry was not tested by a human; please review it before merging.

Discovered on GitHub as publishing AppImages on its releases, by
discover-apps.yml. Needs a maintainer's review.
@github-actions github-actions Bot added the error-missing-file Test failed: the AppImage lacks a required file label Oct 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

❌ Test failure

@deepentropy: this pull request adds the AppImage from your GitHub repository to the AppImage catalog; the test reports the following. Could you have a look?

Tested:

Names:

WARNING: AppImage name 'OpenTrader_linux_amd64.AppImage': should not contain 'Linux', since all AppImages are for Linux (e.g. 'App-1.0-x86_64.AppImage', not 'App-1.0-linux-x86_64.AppImage')

First error in the log:

FATAL: .DirIcon is missing in /tmp/.mount_runtimLMMeGL

What next: once the AppImage is fixed (e.g. in a new release), comment /retest here to test it again. If it should not be in the catalog after all, close this pull request; an app that is already in the catalog can be removed by a maintainer with /remove.

Commit 5cb2b31. Full log

@deepentropy

deepentropy commented Oct 18, 2026 •

Copy link
Copy Markdown

/retest

@probonopd probonopd closed this Oct 18, 2026
@probonopd probonopd reopened this Oct 18, 2026
@github-actions github-actions Bot removed the error-missing-file Test failed: the AppImage lacks a required file label Oct 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ Test passed

Please check that the screenshot shows the application's main window.

OpenTrader

OpenTrader

Tested:

Compatibility: not self-contained: uses the C library of the system; references glibc 2.35

Names:

WARNING: AppImage name 'OpenTrader_linux_amd64.AppImage': should not contain 'Linux', since all AppImages are for Linux (e.g. 'App-1.0-x86_64.AppImage', not 'App-1.0-linux-x86_64.AppImage')

Warnings:

  • Please check the remarks about the names below.
  • The window is mostly dark, so the application seems to start in a dark theme although the test system is not set to dark mode. Please use a light mode by default and use the dark theme only when the system is configured for dark mode (e.g. the GTK/Qt or theme setting) or the user chooses it in the settings.
  • The AppImage contains no update information, so users cannot update it with AppImageUpdate or similar tools. Please consider embedding it when building the AppImage (e.g., appimagetool -u) and publishing the .zsync file next to the AppImage; see https://docs.appimage.org/packaging-guide/optional/updates.html

Commit 5cb2b31. Full log

@deepentropy

Copy link
Copy Markdown

/retest

@probonopd probonopd closed this Oct 18, 2026
@probonopd probonopd reopened this Oct 18, 2026
@github-actions github-actions Bot added the screenshot-ok Test passed; the screenshot was checked and looks fine label Oct 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ Test passed

Please check that the screenshot shows the application's main window.

OpenTrader

OpenTrader

Tested:

Compatibility: not self-contained: uses the C library of the system; references glibc 2.35

Names:

WARNING: AppImage name 'OpenTrader_linux_amd64.AppImage': should not contain 'Linux', since all AppImages are for Linux (e.g. 'App-1.0-x86_64.AppImage', not 'App-1.0-linux-x86_64.AppImage')

Warnings:

  • Please check the remarks about the names below.
  • The AppImage contains no update information, so users cannot update it with AppImageUpdate or similar tools. Please consider embedding it when building the AppImage (e.g., appimagetool -u) and publishing the .zsync file next to the AppImage; see https://docs.appimage.org/packaging-guide/optional/updates.html

Commit 5cb2b31. Full log

@probonopd
probonopd merged commit 811eeaa into master Oct 18, 2026
10 of 11 checks passed
@probonopd
probonopd deleted the discover/OpenTrader branch October 18, 2026 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-discovered screenshot-ok Test passed; the screenshot was checked and looks fine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants