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

separateMajorMinor=false patch grouping broken since v25.0.0 #9744

Closed
ahippler opened this issue Apr 27, 2021 · 11 comments
Closed

separateMajorMinor=false patch grouping broken since v25.0.0 #9744

ahippler opened this issue Apr 27, 2021 · 11 comments
Labels
status:requirements Full requirements are not yet known, so implementation should not be started type:bug Bug fix of existing functionality

Comments

@ahippler
Copy link
Contributor

What Renovate type, platform and version are you using?

self-hosted bitbucket-server

Describe the bug

Since v25.0.0 it is not possible to combine all changes of a group.

{
    "packageRules": [
        {
            "groupName": "all",
            "packagePatterns": ["*"],
            "separateMinorPatch": false,
            "separateMajorMinor": false,
            "separateMultipleMajor": false
        }
    ]
}

did produce 1 PR before 25.0.0 and after the update there is an additional renovate/patch-all branch.

Relevant debug logs

package.json

{
    "name": "test",
    "version": "0.0.1",
    "devDependencies": {
        "lodash": "4.17.20",
        "sinon": "9.0.0"
    }
}

renovate.json

{
    "packageRules": [
        {
            "groupName": "all",
            "packagePatterns": ["*"],
            "separateMinorPatch": false,
            "separateMajorMinor": false,
            "separateMultipleMajor": false
        }
    ]
}

v25.4.1: DEBUG: Processing 2 branches: renovate/all, renovate/patch-all
v25.0.0: DEBUG: Processing 2 branches: renovate/all, renovate/patch-all
v24.119.11: DEBUG: Processing 1 branch: renovate/all

@ahippler ahippler added priority-5-triage status:requirements Full requirements are not yet known, so implementation should not be started type:bug Bug fix of existing functionality labels Apr 27, 2021
@rarkins rarkins added the auto:reproduction A minimal reproduction is necessary to proceed label Apr 27, 2021
@github-actions
Copy link
Contributor

Hi there,

The Renovate team needs your help! Before we can start work on your issue we first need to know exactly what's causing the current behavior. A minimal reproduction helps us with this.

To get started, please read our guide on minimal reproductions to understand what is needed.

We may close the issue if you have not provided a minimal reproduction within two weeks. If you need more time, or are stuck, please ask for help or more time in a comment.

Good luck,

The Renovate team

@rarkins
Copy link
Collaborator

rarkins commented Apr 27, 2021

Can you put this into a simple public github.com repo that we can fork to test? No need to run any bot against it first

@rarkins rarkins changed the title separateMajorMinor broken since v25.0.0 separateMajorMinor=false patch grouping broken since v25.0.0 Apr 27, 2021
@ahippler
Copy link
Contributor Author

@rarkins rarkins added reproduction:provided and removed auto:reproduction A minimal reproduction is necessary to proceed labels Apr 27, 2021
@github-actions
Copy link
Contributor

Thank you for providing a reproduction! 🎉 🚀

The Renovate team will take a look at the reproduction repository. Once we confirm the provided repository reproduces the problem, the label will be changed to reproduction:confirmed.

@trim21

This comment has been minimized.

@rarkins

This comment has been minimized.

@rarkins

This comment has been minimized.

@trim21
Copy link
Contributor

trim21 commented Apr 27, 2021

@trim21 actually what you are seeing is definitely different. The hosted bot/app is not running v25 yet

No, I'm running renovate cli v25 on this repo. I just update renovate config, remove all extra config.

=========

https://github.com/Trim21/poetry-lock-cannot-update/issues/19 this issue is created by renovate cli v25

This issue contains a list of Renovate updates and their statuses.

Awaiting Schedule

These updates are awaiting their schedule. Click on a checkbox to ignore the schedule.

  • chore(deps): update deps-prod (patch) (sphinx, sphinx-rtd-theme)
  • chore(deps-dev): update deps-dev (patch) (pytest, pytest-github-actions-annotate-failures)
  • chore(deps-dev): update deps-dev (flake8, flake8-bugbear, flake8-comprehensions, pre-commit)

Edited/Blocked

These updates have been manually edited so Renovate will no longer make changes. To discard all commits and start over, check the box below.

  • chore(deps): pin dependencies (requests, typing_extensions)

@renovate-release
Copy link
Collaborator

🎉 This issue has been resolved in version 25.4.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@viceice
Copy link
Member

viceice commented Apr 27, 2021

@trim21 Please try v25.4.3, which was just released.

@trim21
Copy link
Contributor

trim21 commented Apr 27, 2021

@viceice fixed, thanks

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:requirements Full requirements are not yet known, so implementation should not be started type:bug Bug fix of existing functionality
Projects
None yet
Development

No branches or pull requests

5 participants