Change cs's style and make js be oop#6
Change cs's style and make js be oop#6shyenjian wants to merge 2 commits intoSHELTER-ZONE:masterfrom
Conversation
HTGAzureX1212
left a comment
HTGAzureX1212
left a comment
There was a problem hiding this comment.
All changes involved in C#, are basically invalid as the method names in C#, as a naming convention, should be PascalCase.
|
Therefore I am fully against the commits on the C# part. |
By that I mean the changes are against the naming convention. |
HTGAzureX1212
left a comment
HTGAzureX1212
left a comment
There was a problem hiding this comment.
Naming Conventions fixed.
C#_Cert.cs
Outdated
| } | ||
|
|
||
| public void Run(Func<string, string> function) | ||
| public void run(Func<string, string> function) |
There was a problem hiding this comment.
Invalid: Methods should be PascalCase under the C# Naming Convention.
C#_Cert.cs
Outdated
| } | ||
|
|
||
| public string GetPassword(string userID) | ||
| public string getPassword(string userID) |
There was a problem hiding this comment.
Invalid: Method names should be PascalCase under the C# Naming Convention.
C#_Cert.cs
Outdated
| } | ||
|
|
||
| public static long Encrypt(long number) => | ||
| public static long encrypt(long number) => |
There was a problem hiding this comment.
Invalid: Method names should be PascalCase under the C# Naming Convention
C#_Cert.cs
Outdated
| Convert.ToInt64(CheckType(typeof(long), number)) >> 23 ^ 2333; | ||
|
|
||
| static string Solve(string token) | ||
| static string solve(string token) |
There was a problem hiding this comment.
Invalid: Method names should be PascalCase under the C# Naming Convention.
|
js is still not oop 🤔 |
|
Where is |
3 participants
No description provided.