Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update skeleton dependencies patch and some feature releases #31 #32

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

StorytellerCZ
Copy link
Member

No description provided.

@StorytellerCZ StorytellerCZ requested review from dd137 and a4xrbj1 October 4, 2023 13:39
@a4xrbj1 a4xrbj1 merged commit f76405a into fix/bug-12 Oct 4, 2023
@a4xrbj1
Copy link

a4xrbj1 commented Oct 4, 2023

@StorytellerCZ - I have Winston at version 3.3.0 (first version with Node 14) and it works fine it seems (I haven't tested it on Production, so I'm not able to see if the logging still works fine with DataDog which is my aggregator of log files.

I couldn't find version 2.7.7 though.

Also note that I'm still using reify 0.17.3 and not meteor/reify - but again no problems with the app from the limited testing I did.

@a4xrbj1
Copy link

a4xrbj1 commented Oct 4, 2023

@dd137 - in regards to your comment that this package.json is only used for the skeleton app. Can confirm that however starting my app with the debug option shows that the electronAppScaffold is overwriting any changes and reverts to the original values:

VERBOSE meteorApp: updating .meteor/.gitignorez
VERBOSE meteorApp: desktopHCP is not enabled, removing required packages
VERBOSE electronAppScaffold: clearing desktop-build
VERBOSE electronAppScaffold: creating /Users/andreaswest/Documents/workspace/frontend/.meteor/desktop-build/app
VERBOSE electronAppScaffold: copying skeleton app
VERBOSE electronAppScaffold: writing package.json
VERBOSE electronApp: updating package.json fields
DEBUG electronApp: writing updated package.json
INFO electronApp: updating list of package.json's dependencies
DEBUG electronApp: merging settings.json[dependencies]
DEBUG electronApp: merging settings.json[plugins]
DEBUG electronApp: merging dependencies from modules
DEBUG electronApp: writing updated package.json
VERBOSE electronApp: calculating compatibility version
DEBUG electronApp: meteor-desktop compatibility version is 3
DEBUG electronApp: .desktop compatibility version is 3
INFO electronApp: issuing node_modules install from electron-builder

@a4xrbj1
Copy link

a4xrbj1 commented Oct 4, 2023

Integration test fails here:

=> Errors while parsing arguments:

While adding package communitypackages:meteor-desktop-watcher@=1.7.0:
error: no such version communitypackages:[email protected]

While adding package communitypackages:meteor-desktop-bundler@=1.7.0:
error: no such version communitypackages:[email protected]

ERROR electronApp: error while checking for required packages: Error: Error: adding packages failed
at MeteorApp.ensureDesktopHCPPackages (/home/runner/work/meteor-desktop/meteor-desktop/lib/meteorApp.js:83:23)
at processTicksAndRejections (internal/process/task_queues.js:95:5)
at ElectronApp.build (/home/runner/work/meteor-desktop/meteor-desktop/lib/electronApp.js:316:13)
at MeteorDesktop.build (/home/runner/work/meteor-desktop/meteor-desktop/lib/index.js:131:9)
at Context. (/home/runner/work/meteor-desktop/meteor-desktop/tests/integration/integration.test.js:239:13)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @meteor-community/[email protected] test-integration: mocha -t 10000 -r @meteorjs/reify "tests/+(integration)/**/*.test.js"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @meteor-community/[email protected] test-integration script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR! /home/runner/.npm/_logs/2023-10-04T15_35_13_451Z-debug.log
Error: Process completed with exit code 1.

I'm not running HCP as it was causing errors with my app, so no idea what's wrong here.

@dd137
Copy link

dd137 commented Oct 5, 2023

Ah, I just noticed some of the same problems (#13 (comment)).

@a4xrbj1 I see. Looks indeed like package.json is rewritten every time. That explain you initial observation. Let's update the dependencies in skeletonDependencies.js, as per this PR and your issue 👍

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

Successfully merging this pull request may close these issues.

3 participants