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

Renovate not updating pubspec.lock #6049

Closed
2 tasks done
rohansingh opened this issue Apr 24, 2020 · 11 comments · Fixed by #19116
Closed
2 tasks done

Renovate not updating pubspec.lock #6049

rohansingh opened this issue Apr 24, 2020 · 11 comments · Fixed by #19116
Assignees
Labels
manager:pub priority-3-medium Default priority, "should be done" but isn't prioritised ahead of others status:in-progress Someone is working on implementation type:feature Feature (new functionality)

Comments

@rohansingh
Copy link
Contributor

rohansingh commented Apr 24, 2020

What Renovate type are you using?

Self-hosted, with --binary-source docker.

Describe the bug

Renovate is opening pull requests that update pubspec.yaml, but which do not update pubspec.lock with corresponding changes.

Did you see anything helpful in debug logs?

Nothing specific, but here is the generation of one PR, to upgrade the dependency flutter_typeahead:

Logs
DEBUG: processBranch with 1 upgrades (repository=tidbyt/tidbyt, dependencies=flutter_typeahead, branch=renovate/flutter_typeahead-1.x)
--
  | DEBUG: Setting baseBranch to master (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: latest commit (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | "branchName": "master",
  | "latestCommitDate": "2020-04-24 16:30:42 -0400"
  | DEBUG: getBranchPr(renovate/flutter_typeahead-1.x) (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: findPr(renovate/flutter_typeahead-1.x, undefined, open) (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: branchExists=false (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Branch has been checked in master issue: unlimit (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Branch has 1 upgrade(s) (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: recreateClosed is false (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: findPr(renovate/flutter_typeahead-1.x, fix(deps): update dependency flutter_typeahead to v1.8.0, !open) (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: prAlreadyExisted=false (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Checking schedule(at any time, null) (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: No schedule defined (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Branch needs creating (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Using parentBranch: undefined (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: manager.getUpdatedPackageFiles() (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Starting search at index 861 (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Found match at index 861 (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Contents updated (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Updated 1 package files (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Getting updated lock files (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: lock file dirs (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | "dirs": {
  | "yarnLockDirs": [],
  | "npmLockDirs": [],
  | "pnpmShrinkwrapDirs": [],
  | "lernaDirs": []
  | }
  | DEBUG: Writing package.json files (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | "packageFiles": ["package.json"]
  | DEBUG: Writing package-lock.json (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Writing any updated package files (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: No updated lock files in branch (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Checking for post-upgrade tasks (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | "tasks": {"commands": [], "fileFilters": []},
  | "allowedCommands": [".*"]
  | DEBUG: 1 file(s) to commit (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)
  | DEBUG: Committing files to branch renovate/flutter_typeahead-1.x (repository=tidbyt/tidbyt, branch=renovate/flutter_typeahead-1.x)

To Reproduce

This is a private monorepo but I'm happy to provide access temporarily if it helps with debugging.

Additional context

This is a monorepo with the Dart project living in the flutter/ directory, so the relevant files are flutter/pubspec.yaml and flutter/pubspec.lock. Perhaps this is similar to #2519?

blocked by

@rarkins
Copy link
Collaborator

rarkins commented Apr 24, 2020

Looks like we don't actually have pubspec.lock support yet. If you can set up a demo public repo and find any relevant links, it will help to getting this feature ready.

@rarkins rarkins added type:feature Feature (new functionality) needs-requirements priority-3-medium Default priority, "should be done" but isn't prioritised ahead of others labels Apr 24, 2020
@viceice
Copy link
Member

viceice commented Apr 25, 2020

https://dart.dev/tools/pub/cmd/pub-upgrade

We need to run pub upgrade xyz for single update and pup upgrade to update all dep in pupspec.yml range.

@rarkins can you prepare docker-pub and maybe docker-dart repo or do we have it?

@rarkins
Copy link
Collaborator

rarkins commented Apr 25, 2020

Does it work if we’ve updated the yaml directly first? And can we add multiple package names in one command?

@viceice
Copy link
Member

viceice commented Apr 25, 2020

Yes, we only need docker-dart, pub is part of the sdk. pub upgrade simply ignores the lockfile and creates a new file with updates in range. It never touches the pubspec.yml.

it also supports multiple upgrades: pub upgrade xyz abc def

@viceice
Copy link
Member

viceice commented Apr 25, 2020

With dep name passed it only upgrades the dep and transitive deps (only the lockfile is updated)

@rarkins rarkins added the status:requirements Full requirements are not yet known, so implementation should not be started label Jan 12, 2021
@zeshuaro

This comment was marked as off-topic.

@viceice

This comment was marked as resolved.

@viceice viceice added status:blocked Issue is blocked by another issue or external requirement and removed status:requirements Full requirements are not yet known, so implementation should not be started labels Nov 24, 2021
@viceice viceice added status:in-progress Someone is working on implementation and removed status:blocked Issue is blocked by another issue or external requirement labels Jun 14, 2022
@viceice
Copy link
Member

viceice commented Jun 14, 2022

buildpack and sidecar image prepared, this should now use our sidecar image with buildpack code

const allToolConfig: Record<string, ToolConfig> = {

const execOptions: ExecOptions = {
docker: {
image: 'sidecar',
},
toolConstraints: [
{
toolName: 'flux',
constraint: updatedDeps[0].newVersion,
},
],
};

@viceice

This comment was marked as resolved.

@provokateurin

This comment was marked as resolved.

@renovate-release
Copy link
Collaborator

🎉 This issue has been resolved in version 34.73.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
manager:pub priority-3-medium Default priority, "should be done" but isn't prioritised ahead of others status:in-progress Someone is working on implementation type:feature Feature (new functionality)
Projects
None yet
6 participants