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

Spelling and whitespace fixes #585

Merged
merged 15 commits into from
Dec 14, 2024
Merged

Conversation

SamErde
Copy link
Contributor

@SamErde SamErde commented Dec 13, 2024

A little bit of polish for Maester:

  • Add cSpell settings with project word list and ignored words
  • Import cSpell dictionaries for PowerShell and C# projects
  • Fix multiple typos
  • Remove trailing whitespace and lines that only contain extraneous whitespace

@SamErde SamErde requested review from a team as code owners December 13, 2024 21:41
Copy link
Contributor

@merill merill left a comment

Choose a reason for hiding this comment

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

Awesome. Thanks @SamErde

for future reference the EIDCSA.ps1 files are auto generated so any changes will be overwritten when we run the script in ./build

@merill merill merged commit e7bb8f9 into maester365:main Dec 14, 2024
0 of 4 checks passed
@SamErde SamErde deleted the se-settings-01 branch December 14, 2024 03:13
@SamErde
Copy link
Contributor Author

SamErde commented Dec 14, 2024

Ah, thanks for the reminder! I would like to know more about how that part of the build works. I have also been curious about why they're in a different folder structure than the other tests.

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.

2 participants