Skip to content

The installed-app check can be run - #860

Merged
sridharkalaibala merged 1 commit into
developfrom
fix/the-installed-app-check-can-run
Sep 17, 2026
Merged

sridharkalaibala merged 1 commit into
developfrom
fix/the-installed-app-check-can-run

Conversation

@sridharkalaibala

Copy link
Copy Markdown
Contributor

scripts/check-installed-app.js declares a shebang and was committed 100644; tests/executable-scripts.test.js has failed the desktop suite on develop over it since it landed. This is the mode bit and nothing else. node --test tests/executable-scripts.test.js: 2/2.

scripts/check-installed-app.js declares a shebang and was committed
100644. cron or a deploy that calls it by path gets 'permission denied'.
tests/executable-scripts.test.js has refused the suite over it since it
landed; this is the mode bit, nothing else.
@sridharkalaibala
sridharkalaibala merged commit 6086e67 into develop Sep 17, 2026
9 checks passed
@github-actions github-actions Bot added the ready for QA Merged to develop and live on develop.posnic.io - anyone can test it label Sep 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Merged to develop. Anyone can test this - you do not need write access.

Try it at https://develop.posnic.io, or run it yourself:

git fetch origin develop && git checkout develop
npm install && npm --prefix api install
npm run dev   # then http://localhost:3000

When you have tested it, say what you did and what happened, and set
QA passed or QA failed. If you cannot set labels, just comment -
a maintainer will.

Reporting that something is broken is as useful as fixing it. It is
better found here than by a shopkeeper.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for QA Merged to develop and live on develop.posnic.io - anyone can test it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant