search

adonisjs / transmit

A Server-Sent-Event module for AdonisJS
MIT License
65 stars 4 forks source link

Warning on install the transmit library. #22

Closed oyepez003 closed 3 months ago

oyepez003 commented 3 months ago

Package version

^1.0.1

Describe the bug

On installing the library I got the next warning:

~/app $ node ace add @adonisjs/transmit
[ info ] Installing the package using the following command : npm add @adonisjs/transmit
❯ Continue ? (y/N) · true
[ wait ] package installed successfully ...
SKIPPED: create config/transmit.ts (File already exists)
[ warn ] Cannot update "adonisrc.ts" file. Install "@adonisjs/assembler" to modify source files
[ success ] Installed and configured @adonisjs/transmit

[ warn ] Cannot update "adonisrc.ts" file. Install "@adonisjs/assembler" to modify source files

The process fails to update the adonisrc.ts. The @adonisjs/assembler library is already installed in my project.

I tried to execute the next command too:

~/app $ node ace configure @adonisjs/transmit
SKIPPED: create config/transmit.ts (File already exists)
[ warn ] Cannot update "adonisrc.ts" file. Install "@adonisjs/assembler" to modify source files
~/app $

But I got the same warning message.

To fix the error I've registered the @adonisjs/transmit/transmit_provider inside the adonisrc.ts file.

I've already installed other packages in my project and I never got this warning before.

Thanks for the amazing project AdonisJS and all its ecosystem.

Reproduction repo

No response

oyepez003 commented 3 months ago

I closed this ticket because it was an internal error in my settings. I am working using docker and for some reason when the preloads try to update the file adonisrc.ts it fails when some folder doesn't have the correct permissions. In my case, the permissions on the docker volume throw the error.

Maybe, excluding some folders in the preloads process should be a good thing to be added.