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

refactor: remove hardcoded versions for PlayFabParty and DepotDownloader #55

Merged

Conversation

thesharp
Copy link
Contributor

As you've mentioned in #54, we may need an update for PlayFabParty, so this would make changing the versions for both PlayFabParty and DepotDownloader easier.

@timkurvers
Copy link
Owner

Thanks! 🥳 Could you drop the DepotDownloader bits? There's hopefully no need to differentiate how we download depots from Steam between the stable and beta branches.

@thesharp
Copy link
Contributor Author

thesharp commented Nov 4, 2023

Yeah, I didn't think that there would be need to differentiate how we download depots for both branches. But I thought about changing the version of DepotDownloader sometime due to SteamRE/DepotDownloader#450. And with this it would involve only changing the version and hash once, instead of changing the version 9 times and the hash once.

But if you think that it's unnecessary, I will drop it.

@timkurvers
Copy link
Owner

Ah, that makes a lot of sense, all good! 🥳

@timkurvers timkurvers merged commit d595d1c into timkurvers:main Nov 6, 2023
@thesharp thesharp deleted the feat/remove-hardcoded-versions branch November 6, 2023 10:16
thesharp added a commit to thesharp/valheim-macos that referenced this pull request Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants