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

Code Enhancements #794

Merged
merged 4 commits into from
Mar 8, 2022
+16 −19
Merged

Code Enhancements #794

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
29f5b49
Let private members readonly
WeihanLi Mar 8, 2022
3c833a2
Use long.TryParse
WeihanLi Mar 8, 2022
ecfd154
Sealed internal types
WeihanLi Mar 8, 2022
1032230
Sealed private types
WeihanLi Mar 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Let private members readonly
@WeihanLi
WeihanLi authored Mar 8, 2022

Verified

This commit was signed with the committer’s verified signature.
snyk-bot Snyk bot
GPG key ID: 78AC5AE55A47A65B
Verified
Learn about vigilant mode
commit 29f5b49adf70618ac861756f1d3f641fd64a6d9f
9 changes: 3 additions & 6 deletions src/KubernetesClient/Autorest/StringTokenProvider.cs
View file
Original file line number Diff line number Diff line change
@@ -12,8 +12,8 @@ namespace k8s.Autorest
/// </summary>
public sealed class StringTokenProvider : ITokenProvider
{
private string _accessToken;
private string _type;
private readonly string _accessToken;
private readonly string _type;

/// <summary>
/// Initializes a new instance of the <see cref="StringTokenProvider"/> class.
@@ -31,10 +31,7 @@ public StringTokenProvider(string accessToken, string tokenType)
/// <summary>
/// Gets the token type of this access token.
/// </summary>
public string TokenType
{
get { return _type; }
}
public string TokenType => _type;

/// <summary>
/// Returns the static access token.