Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@wreulicke
Copy link
Contributor

@wreulicke wreulicke commented Feb 27, 2025

Currently, ListArtifacts is used without a name option even though there is in github API reference.

So, I checked the current version of go-github doesn't have that.
But, the latest version of go-github has a name option.

So, this PR upgrade go-github (and the dependants) and pass the name to ListArtifacts.

@wreulicke
Copy link
Contributor Author

Sorry, I will check the failure of CI.

@wreulicke
Copy link
Contributor Author

ah, I checked the failure.
github.com/k1LoW/ghfs requires github.com/google/go-github/v58, hmm.

@wreulicke
Copy link
Contributor Author

wreulicke commented Feb 28, 2025

I also created a PR to ghfs.
This PR requires upgrading go-github in ghfs.
k1LoW/ghfs#34

So, I change to draft

@wreulicke wreulicke marked this pull request as draft February 28, 2025 00:54
@wreulicke wreulicke marked this pull request as ready for review February 28, 2025 14:46
@wreulicke
Copy link
Contributor Author

wreulicke commented Feb 28, 2025

@k1LoW
I updated ghfs to latest. Could you check this PR again?

@wreulicke wreulicke changed the title Upgrade go-github and pass name option Upgrade go-github and github deps and pass name option Feb 28, 2025
@wreulicke wreulicke changed the title Upgrade go-github and github deps and pass name option Upgrade go-github and github dependants and pass name option Feb 28, 2025
@wreulicke wreulicke changed the title Upgrade go-github and github dependants and pass name option Upgrade go-github (and the dependants) and pass name option Feb 28, 2025
@k1LoW k1LoW added the minor label Feb 28, 2025
@k1LoW
Copy link
Owner

k1LoW commented Feb 28, 2025

@wreulicke Could you rebase or update branch? ( crypto has been updated )

@wreulicke
Copy link
Contributor Author

@k1LoW
updated.

@k1LoW
Copy link
Owner

k1LoW commented Feb 28, 2025

@wreulicke Thank you!!!

@k1LoW k1LoW merged commit 901c3b0 into k1LoW:main Feb 28, 2025
5 checks passed
@github-actions github-actions bot mentioned this pull request Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants