Remove OK code when sentinel is matched

[quinnj]

This broke the InlineStrings tests. Regardless of greedy or non-greedy, we want to remove the OK code, since that should be mutually exclusive with the SENTINEL code.

[codecov]

Codecov Report

Base: 90.98% // Head: 90.98% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (fed4606) compared to base (388613c).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #142   +/-   ##
=======================================
  Coverage   90.98%   90.98%           
=======================================
  Files           9        9           
  Lines        1608     1609    +1     
=======================================
+ Hits         1463     1464    +1     
  Misses        145      145           

Impacted Files	Coverage Δ
src/components.jl	99.16% <100.00%> (+<0.01%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[nickrobinson251]

Thanks!

I'm surprised the existing tests here in Parsers.jl did't catch this -- let's add one?

This fails on main but passes with this fix:

# #140, #142 
res = Parsers.xparse(String, "NA"; sentinel=["NA"])
@test res.code == (SENTINEL | EOF)