Skip to content

feat(languages): add support for PowerShell#276

Merged
alstr merged 2 commits intoalstr:masterfrom
RomainChamb:add-powershell-support
Jul 10, 2025
Merged

feat(languages): add support for PowerShell#276
alstr merged 2 commits intoalstr:masterfrom
RomainChamb:add-powershell-support

Conversation

@RomainChamb
Copy link
Copy Markdown

Description

Language Name: PowerShell

Checklist for Adding a New Language

Before submitting your pull request, please ensure the following requirements are met:

  • I have added the language to syntax.json with the language name matching that in GitHub's Linguist languages.yml file.
  • I have added sample code to test_new.diff and test_closed.diff.
  • I have added issue creation and closure tests to test_todo_parser.py. The tests should check that the ace_mode of the issue matches that specified in the languages.yml file. If existing checks for that ace_mode exists, I have incremented them instead.
  • I have updated README.md to add the language.
  • I have run and verified all tests.

@RomainChamb
Copy link
Copy Markdown
Author

Hi guys,
This is my first time ever contribution to a project. Please let me know if I miss something.
I've run the test, there is one which seems to be falling on master. Not sure what to do with it.
Thank you for your help.

@alstr
Copy link
Copy Markdown
Owner

alstr commented Jul 9, 2025

Thanks for the PR, it all looks good to me from here, I think the test is just failing because of the hardcoded number we have in test_process_diff.py:

self._standardTest(84)

I think we just need to update that to 87 to account for the new TODOs added to the test file.

This is a bit unclear as it stands, so I will make a note to mention it in the README and PR template. Thanks!

@RomainChamb
Copy link
Copy Markdown
Author

Thank you for your message.
I've just updated the test as you mentioned.

@alstr alstr merged commit 7f4e839 into alstr:master Jul 10, 2025
1 check passed
@alstr
Copy link
Copy Markdown
Owner

alstr commented Jul 10, 2025

Thanks!

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