invertase / melos

🌋 A tool for managing Dart projects with multiple packages. With IntelliJ and Vscode IDE support. Supports automated versioning, changelogs & publishing via Conventional Commits.
https://melos.invertase.dev/~melos-latest
Apache License 2.0
1.11k stars 198 forks source link

request: bump SDK version > 3.1.0 #605

Closed sebastianbuechler closed 8 months ago

sebastianbuechler commented 9 months ago

Is there an existing feature request for this?

Command

No response

Description

When using the latest version of the lint package melos is incompatible:

Because my_workspace depends on lint >=2.2.0 which requires SDK version >=3.1.0 <4.0.0, version solving failed.

Reasoning

Keeping dependencies up to date.

Additional context and comments

No response

spydon commented 9 months ago

The lint package should preferably be depended on a package level, not in the root pubspec.yaml.

packages/
|__ your_package/
   |__ pubspec.yaml <-- in here
pubspec.yaml <-- Only melos version and sdk version
spydon commented 8 months ago

@sebastianbuechler here is a good example of how the directory structure can look like to not have a package/app in the root: https://github.com/invertase/melos/issues/616#issuecomment-1856559131

I'll close this meanwhile since Melos doesn't really support having a package or app at the root of the monorepo.

sebastianbuechler commented 8 months ago

@spydon I don't have an app in root but specify the workspace there with a melos config file. The issue still persists.

spydon commented 8 months ago

@spydon I don't have an app in root but specify the workspace there with a melos config file. The issue still persists.

Can you post your root pubspec?

sebastianbuechler commented 7 months ago

@spydon I don't have an app in root but specify the workspace there with a melos config file. The issue still persists.

Can you post your root pubspec?

Sorry for the late reply. Here it is:

name: company_workspace

environment:
  sdk: '>=3.0.0 <4.0.0'
dev_dependencies:
  melos: ^4.0.0
  lint: ^2.1.2
spydon commented 7 months ago

@sebastianbuechler the lint package shouldn't be in there, it should be in the packages pubspec files.