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

Break the fucking old world with type bundle in package json #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

protiev
Copy link
Collaborator

@protiev protiev commented Jun 10, 2024

doesnt work but just saving the weekend failed state

protiev added 5 commits June 9, 2024 07:31
npx jsr add @axhxrx/date
Installing @axhxrx/date...
$ npm install @axhxrx/date@npm:@jsr/axhxrx__date

added 1 package, and audited 618 packages in 866ms

81 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

Completed in 939ms
…sn't work[1])

[1]: nx run node-cli-app:build

> nx run node-cli-app:build:production

apps/node-cli-app/src/NodeCliApp.ts:1:49 - error TS2307: Cannot find module '@axhxrx/date' or its corresponding type declarations.
  There are types at '/home/protiev/axhxrx-nx-deno-jsr-integration-test-2024-06/node_modules/@axhxrx/date/_dist/mod.d.ts', but this result could not be resolved under your current 'moduleResolution' setting. Consider updating to 'node16', 'nodenext', or 'bundler'.

> 1 | import { dateToIS08601WithTimeZoneOffset } from '@axhxrx/date';
    |                                                 ^
  2 | export class NodeCliApp
  3 | {
  4 |   constructor(

Found 1 error.

——————————————————————————————————————————————————————————————————————————————————

 NX   Ran target build for project node-cli-app (481ms)

   ✖  1/1 failed
   ✔  0/1 succeeded [0 read from cache]

┌[protiev@fed-40-container] [/dev/pts/7] [BREAK-THE-FUCKING-OLD-WORLD-with-type-bundle-in-package-json ⚡] [1]
└[~/axhxrx-nx-deno-jsr-integration-test-2024-06]>
https://github.com/axhxrx/axhxrx-nx-esm-fuckery work to some extent, maybe this could be useful for testing stuff before deleting the repo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant