You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Update to pod2 repo with prefixless wildcards (#123)
PODLang breaking change removes ? from wildcards. This PR updates the
pod2 repo pointer to the latest, and updates wildcards in this repo to
the new format.
I did the replacement with a regex in VSCode. Unlike in the pod2 repo, I
had to tweak it to avoid several classes of false-positives of things
which look like PODLang wildcards but aren't. Documenting the regex I
used for posterity, so that others can apply it on their own branches:
regex:
`(?<!\w|trace(?:.|\n){0,100}|debug(?:.|\n){0,100}|https?://(?:.|\n){0,100})\?([A-Za-z]\w*)(?!\w|=)`
replace with: `$1`
exclude files: `*.cjs`
As far as I can tell from local Rust tests and the GitHub workflow tests
above, this seems to be correct.
I don't know how to build and run the PODNet Client myself, though, so I
haven't tested beyond what's automated.
I'm not expecting a line-by-line review here since it's all automated.
I'm hoping @robknight can quickly stanity-check any false-positives I
may have missed, and confirm the client isn't entirely broken.
0 commit comments