Skip to content

Conversation

@Cameron-Low
Copy link
Contributor

@Cameron-Low Cameron-Low commented Oct 25, 2024

  • ^match pattern now allowed within a code pos
  • #cname can be used to select the appropriate sub-branch of a match
  • ^lv<@ and ^lv<$ are now permited

@Cameron-Low Cameron-Low requested a review from strub October 25, 2024 14:27
@Cameron-Low Cameron-Low self-assigned this Oct 25, 2024
@strub
Copy link
Member

strub commented Oct 29, 2024

If you don't mind, making two PR (you make two things in that PR) would make more sense. (I'd like to use the PR for generating a changelog)

@Cameron-Low
Copy link
Contributor Author

Removed the new print command from this PR.

@strub
Copy link
Member

strub commented Oct 30, 2024

Could you add an example? And then, we'll be good to go.

@Cameron-Low Cameron-Low force-pushed the zipper-upgrade branch 2 times, most recently from f12f20d to c0c2e48 Compare October 30, 2024 12:27
@Cameron-Low
Copy link
Contributor Author

I've added the test, and made minor changes to pp_codepos to account for the recent changes to main.

@strub strub changed the title [Internal] Allow Match in Code Positions Improve code positions (match + extended assignments) Oct 30, 2024
 - `^match` pattern now allowed within a code pos
 - `#cname` can be used to select the appropriate sub-branch of a match,
   e.g. `^match#Some.1`
 - `^lv<@` and `^lv<$` are now permitted
@Cameron-Low Cameron-Low merged commit 9eaff01 into main Oct 30, 2024
15 checks passed
@Cameron-Low Cameron-Low deleted the zipper-upgrade branch October 30, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants