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

Update console and indicatif deps #20998

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Update console and indicatif deps #20998

merged 1 commit into from
Jun 4, 2024

Conversation

huonw
Copy link
Contributor

@huonw huonw commented Jun 4, 2024

These dependencies were using a git dependency due to unmerged bug-fixes. These fixes have now been merged and are available in the latest releases:

In addition, I think the commits in question may've disappeared? E.g. the fresh build on the new CI platform in had this error: #20997

    Updating git repository `https://github.com/tgolsson/console.git`
error: failed to load source for dependency `console`

Caused by:
  Unable to update https://github.com/tgolsson/console.git?rev=5483880905f384679d322e83c37180f122951995#54838809

Caused by:
  revspec '5483880905f384679d322e83c37180f122951995' not found; class=Reference (4); code=NotFound (-3)

Thus, this is marked for cherry-picking back to our active branches.

@huonw huonw added needs-cherrypick category:internal CI, fixes for not-yet-released features, etc. labels Jun 4, 2024
@huonw huonw added this to the 2.20.x milestone Jun 4, 2024
@huonw huonw requested review from benjyw, tdyas and tgolsson June 4, 2024 07:11
Copy link
Contributor

@tgolsson tgolsson left a comment

Choose a reason for hiding this comment

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

Nice! I'm not sure about the commits disappearing, I've not touched those repos since I added the patches.

@huonw
Copy link
Contributor Author

huonw commented Jun 4, 2024

Weird! Maybe this won't fix the issue in #20997, but I imagine it can't hurt.

@huonw huonw merged commit cc5577e into main Jun 4, 2024
25 checks passed
@huonw huonw deleted the huonw/console-dep branch June 4, 2024 22:03
WorkerPants pushed a commit that referenced this pull request Jun 4, 2024
These dependencies were using a git dependency due to unmerged
bug-fixes. These fixes have now been merged and are available in the
latest releases:

- console-rs/console#186
- console-rs/indicatif#608

In addition, I think the commits in question may've disappeared? E.g.
the fresh build on the new CI platform in had this error:
#20997

```
    Updating git repository `https://github.com/tgolsson/console.git`
error: failed to load source for dependency `console`

Caused by:
  Unable to update https://github.com/tgolsson/console.git?rev=5483880905f384679d322e83c37180f122951995#54838809

Caused by:
  revspec '5483880905f384679d322e83c37180f122951995' not found; class=Reference (4); code=NotFound (-3)
```

Thus, this is marked for cherry-picking back to our active branches.
WorkerPants pushed a commit that referenced this pull request Jun 4, 2024
These dependencies were using a git dependency due to unmerged
bug-fixes. These fixes have now been merged and are available in the
latest releases:

- console-rs/console#186
- console-rs/indicatif#608

In addition, I think the commits in question may've disappeared? E.g.
the fresh build on the new CI platform in had this error:
#20997

```
    Updating git repository `https://github.com/tgolsson/console.git`
error: failed to load source for dependency `console`

Caused by:
  Unable to update https://github.com/tgolsson/console.git?rev=5483880905f384679d322e83c37180f122951995#54838809

Caused by:
  revspec '5483880905f384679d322e83c37180f122951995' not found; class=Reference (4); code=NotFound (-3)
```

Thus, this is marked for cherry-picking back to our active branches.
@WorkerPants
Copy link
Member

I tried to automatically cherry-pick this change back to each relevant milestone, so that it is available in those older releases of Pants.

✔️ 2.20.x

Successfully opened #21001.

✔️ 2.21.x

Successfully opened #21002.


Thanks again for your contributions!

🤖 Beep Boop here's my run link

huonw added a commit that referenced this pull request Jun 5, 2024
These dependencies were using a git dependency due to unmerged
bug-fixes. These fixes have now been merged and are available in the
latest releases:

- console-rs/console#186
- console-rs/indicatif#608

In addition, I think the commits in question may've disappeared? E.g.
the fresh build on the new CI platform in had this error:
#20997

```
    Updating git repository `https://github.com/tgolsson/console.git`
error: failed to load source for dependency `console`

Caused by:
  Unable to update https://github.com/tgolsson/console.git?rev=5483880905f384679d322e83c37180f122951995#54838809

Caused by:
  revspec '5483880905f384679d322e83c37180f122951995' not found; class=Reference (4); code=NotFound (-3)
```

Thus, this is marked for cherry-picking back to our active branches.

Co-authored-by: Huon Wilson <huon@exoflare.io>
huonw added a commit that referenced this pull request Jun 5, 2024
These dependencies were using a git dependency due to unmerged
bug-fixes. These fixes have now been merged and are available in the
latest releases:

- console-rs/console#186
- console-rs/indicatif#608

In addition, I think the commits in question may've disappeared? E.g.
the fresh build on the new CI platform in had this error:
#20997

```
    Updating git repository `https://github.com/tgolsson/console.git`
error: failed to load source for dependency `console`

Caused by:
  Unable to update https://github.com/tgolsson/console.git?rev=5483880905f384679d322e83c37180f122951995#54838809

Caused by:
  revspec '5483880905f384679d322e83c37180f122951995' not found; class=Reference (4); code=NotFound (-3)
```

Thus, this is marked for cherry-picking back to our active branches.

Co-authored-by: Huon Wilson <huon@exoflare.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:internal CI, fixes for not-yet-released features, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants