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

Separate new() and new_with_app_id() #841

Merged
merged 4 commits into from
Jan 26, 2025

Conversation

NathanTaylorHunt
Copy link
Contributor

@NathanTaylorHunt NathanTaylorHunt commented Jan 26, 2025

If you call init() on the SteamAPI, it will automatically determine the AppId, otherwise you can call init_app() and provide an explicit AppId.

https://docs.rs/steamworks/latest/steamworks/struct.Client.html#method.init

@@ -31,6 +31,18 @@ impl SteamworksClient {
}
}

/// Creates and initializes the Steamworks client. This must only be called
/// once per application run.
Copy link
Owner

Choose a reason for hiding this comment

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

Can you add a comment that the app_id will usually be read from a steam_appid.txt file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, I copied the comment from the steamworks rs docs:
https://docs.rs/steamworks/latest/steamworks/struct.Client.html#method.init

Copied from the steamworks rs docs.
@cBournhonesque
Copy link
Owner

Thanks!

@cBournhonesque cBournhonesque added the A-Transport Related to the transport layer label Jan 26, 2025
@cBournhonesque cBournhonesque merged commit 1d1a3e9 into cBournhonesque:main Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Transport Related to the transport layer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants