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

Pascal Casing the structure names to align with the naming for all other types #890

Merged
merged 1 commit into from
Feb 28, 2025

Conversation

MariusNi
Copy link
Contributor

Description

Pascal Casing the structure names to align with the naming for all other types:

PERF_CLOCK
BASELINE_RULE
COMMAND_STEP
REPORTED_PROPERTY
OS_DISTRO_INFO
PATH_LOCATIONS
PASSWORD_CREATION_REQUIREMENTS
SIMPLIFIED_USER
SIMPLIFIED_GROUP
OSCONFIG_LOG_HANDLE
OSCONFIG_LOG

Please comment if you spot any other offenders.

Checklist

  • I have read the contribution guidelines.
  • All unit tests are passing.
  • I have merged the latest dev branch prior to this PR submission.
  • I ran pre-commit on my changes prior to this PR submission.
  • I submitted this PR against the dev branch.

@MariusNi MariusNi requested a review from a team as a code owner February 28, 2025 18:11
@MariusNi MariusNi requested review from robertwoj-microsoft, kkanas and wupeka and removed request for robertwoj-microsoft February 28, 2025 18:12
Copy link

Test Results

287 tests   287 ✅  53s ⏱️
 13 suites    0 💤
  1 files      0 ❌

Results for commit 861182c.

Copy link

github-actions bot commented Feb 28, 2025

Test Results

 44 files   44 suites   37m 2s ⏱️
  4 tests   4 ✅  0 💤 0 ❌
176 runs  154 ✅ 22 💤 0 ❌

Results for commit 861182c.

♻️ This comment has been updated with latest results.

@MariusNi MariusNi merged commit 3f9ca29 into dev Feb 28, 2025
107 checks passed
@MariusNi MariusNi deleted the MariusNi/OsConfig_StructNames_Feb28_2025 branch February 28, 2025 21:55
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.

3 participants