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.

Sign up for GitHub

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

Jump to bottom

Allow authentication in Update-SteamServer #17

Merged
merged 8 commits into from
Jul 6, 2020
Merged

Allow authentication in Update-SteamServer #17

merged 8 commits into from
Jul 6, 2020

Conversation

hjorslev
Copy link
Owner

@hjorslev hjorslev commented Jul 6, 2020

PR Summary

Resolves #16

This PR implements a -Credential parameter in Update-SteamServer allowing to install apps that requires authentication.

Context

Changes

  • Add -Credential parameter
  • Output ExitCode from SteamCMD if it is not 0
  • Refactor code that handles the Credential in Update-SteamApp.

Checklist

  • Pull Request has a meaningful title.
  • Summarised changes.
  • Pull Request is ready to merge & is not WIP.
  • Added tests / only testable interactively.
    • Make sure you add a new test if old tests do not effectively test the code changed.
  • Added documentation / opened issue to track adding documentation at a later date.

@hjorslev hjorslev self-assigned this Jul 6, 2020
@hjorslev hjorslev merged commit 3b3b433 into master Jul 6, 2020
@hjorslev hjorslev deleted the cred branch July 6, 2020 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@hjorslev hjorslev

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Login Info
1 participant
@hjorslev