Skip to content

Conversation

@amirhmoradi
Copy link

What is this PR about?

This updates the build tools to their latest stable version.

Checklist

Before submitting this PR, please make sure that:

Issues related (if applicable)

Screenshots (if applicable)

- pnpm: 9.12.0 → 10.26.0
- Docker: 28.5.2 → 29.1.3
- Nixpacks: 1.39.0 → 1.41.0
- Railpack: 0.2.2 → 0.15.1
- buildpacks/pack: 0.35.0 → 0.39.1
Update to latest stable versions:
- pnpm: 9.12.0 → 10.26.0
- Docker: 28.5.0/28.5.2 → 29.1.3
- Nixpacks: 1.39.0 → 1.41.0
- Railpack: 0.2.2/0.15.0 → 0.15.1
- buildpacks/pack: 0.35.0 → 0.39.1

Files updated:
- All Dockerfiles (main, schedule, cloud, server)
- All package.json files (root, server, api, schedules, dokploy)
- GitHub workflow (pull-request.yml)
- Server setup script
- Database schema and DBML files
- Test fixtures
- UI components
Copy link
Contributor

@Siumauricio Siumauricio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, I appreciate this PR, but updating to newer versions of Docker can cause problems. Versions of packages such as Docker or pnpm are managed by the dokploy team when deemed necessary, when there is a vulnerability, or when it is the right time to update. If you wish, you can remove those changes, and I can accept the update from nixpacks and railpack.

@amirhmoradi
Copy link
Author

Hello, I appreciate this PR, but updating to newer versions of Docker can cause problems. Versions of packages such as Docker or pnpm are managed by the dokploy team when deemed necessary, when there is a vulnerability, or when it is the right time to update. If you wish, you can remove those changes, and I can accept the update from nixpacks and railpack.

@Siumauricio Hi, thanks for the feedback. I have removed those changes as requested.

The reasons I previously introduced those upgrades were:

.default("idle"),
buildType: buildType("buildType").notNull().default("nixpacks"),
railpackVersion: text("railpackVersion").default("0.2.2"),
railpackVersion: text("railpackVersion").default("0.15.1"),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to generate a new migration

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.

3 participants