Skip to content

Conversation

Gijsreyn
Copy link
Contributor

@Gijsreyn Gijsreyn commented Aug 10, 2025

PR Summary

This pull request adds the array(), first(), and indexOf() functions.

PR Context

Partially addresses #57

Note

Everytime I use the build.ps1 locally, those tree-sitter-dscexpression files are added.

@Gijsreyn Gijsreyn force-pushed the add-array-functions branch from d5f37ae to 52cab39 Compare August 11, 2025 23:07
@SteveL-MSFT
Copy link
Member

@Gijsreyn can you resolve the merge conflict then I'll review this. Thanks!

@Gijsreyn
Copy link
Contributor Author

@Gijsreyn can you resolve the merge conflict then I'll review this. Thanks!

Oops, didn't see it had a merge conflict. Should be resolved now!

@Gijsreyn
Copy link
Contributor Author

Just patched up your comments @SteveL-MSFT, but looks like there is an authentication issue.

@SteveL-MSFT
Copy link
Member

@Gijsreyn not an authn issue directly, but there's a new version of serde_json that isn't in our feed, so I just need to add it

@SteveL-MSFT SteveL-MSFT added this pull request to the merge queue Aug 19, 2025
Merged via the queue into PowerShell:main with commit bdc62b1 Aug 19, 2025
7 of 13 checks passed
@Gijsreyn Gijsreyn deleted the add-array-functions branch August 20, 2025 04:48
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