NuKeeper icon indicating copy to clipboard operation
NuKeeper copied to clipboard

NuKeeper is reverting my other changes in it's pull requests.

Open MarkFl12 opened this issue 3 years ago • 7 comments

🐛 Bug Report

I'm running it in TeamCity using a powershell command and quite a few times now it's put in a pull request with the package upgraded but a selection of other recent changes reverted out.

I'm using the command: nukeeper repo https://company.visualstudio.com/CompanyName/_git/ProjectName %GitAccessToken% --maxpackageupdates 500 --exclude "^(AWSSDK|Microsoft|Hangfire|Amazon)\." --age=0 --verbosity detailed --checkout-directory C:\nk\%build.number%all\

Expected behavior

That it upgrades my packages and doesn't undo other work.

Reproduction steps

I can't see anything different about when it does this compared to when it doesn't even with --verbosity detailed. Cancelling the PR and deleting the branch followed by re-running will usually not cause the issue again.

Configuration

Version: 0.34.0

Platform if applicable:

  • [x] :hammer_and_wrench: NuKeeper CLI
  • [x] Windows
  • [x] TeamCity

MarkFl12 avatar Aug 09 '21 08:08 MarkFl12

Can you provide a repro?

msallin avatar Sep 20 '21 17:09 msallin

That makes it happen every time? I'm not sure, I could try running it until the issue happens to get some logs?

MarkFl12 avatar Sep 21 '21 07:09 MarkFl12

It does not happen every time? Could it be that the branch you are creating the PR from is behind?

msallin avatar Sep 21 '21 07:09 msallin

It might well be that the branch is behind, however the checkout directory doesn't even exist until NuKeeper creates it so if this is the case, it's NuKeeper itself that's pulling already behind.

MarkFl12 avatar Sep 21 '21 08:09 MarkFl12

Do you have the chance to check the directory which was pulled by NuKeeper after the problem happened? I.e. check the commit hash and verify that the correct state was pulled.

msallin avatar Sep 21 '21 08:09 msallin

I might well be able to, leave it with me.

MarkFl12 avatar Sep 21 '21 13:09 MarkFl12

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Dec 21 '21 03:12 stale[bot]