Skip to content

chore: migrate to esm#3887

Open
kamilmysliwiec wants to merge 7 commits into
masterfrom
chore/esm-migration
Open

chore: migrate to esm#3887
kamilmysliwiec wants to merge 7 commits into
masterfrom
chore/esm-migration

Conversation

@kamilmysliwiec
Copy link
Copy Markdown
Member

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

# Conflicts:
#	lib/decorators/index.ts
#	lib/explorers/api-response.explorer.ts
#	lib/swagger-explorer.ts
#	lib/swagger-module.ts
#	lib/swagger-scanner.ts
#	lib/swagger-transformer.ts
@Alchez
Copy link
Copy Markdown

Alchez commented May 14, 2026

@kamilmysliwiec can this PR be merged or atleast receive an alpha release in tandem with nestjs/nest#16391 (comment)?

I'm seeing the following trace with the latest version of this package, but the changes in this PR can solve this issue:

> ts-node ./node_modules/typeorm/cli migration:run -d dist/libs/common/src/config/database.config.js

Error during migration run:
Error: Cannot find module '/node_modules/.pnpm/@nestjs+swagger@11.4.2_@nestjs+common@12.0.0-alpha.4_class-transformer@0.5.1_class-vali_328be75e19f9d8897677c1bedae7bfac/node_modules/@nestjs/common/interfaces.js'
    at CommandUtils.loadDataSource (/node_modules/.pnpm/typeorm@0.3.29_ioredis@5.10.1_pg@8.20.0_ts-node@10.9.2_@swc+core@1.15.33_@types+node@25.7.0_typescript@5.9.3_/node_modules/src/commands/CommandUtils.ts:21:19)
    at async Object.handler (/node_modules/.pnpm/typeorm@0.3.29_ioredis@5.10.1_pg@8.20.0_ts-node@10.9.2_@swc+core@1.15.33_@types+node@25.7.0_typescript@5.9.3_/node_modules/src/commands/MigrationRunCommand.ts:42:26)

@kamilmysliwiec
Copy link
Copy Markdown
Member Author

@Alchez i will - as soon as it's ready to be published

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.

2 participants