-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add flag to remove origin from fields #9
Comments
Sounds great! I'll see about shipping that later this evening. |
What I'll do is merge in the PR first for a release. Need to update changelog and push CI for playwright testing, and then can see about filtering the fields afterwards. Update: I'll need a few more days, would love to get some CI going. The old tests didn't do anything but validate a redundant builder pattern...wip. |
Update: Still WIP. Working on a test harness. |
Took way too long to get here. I've decided to skip the test harness, for now. Manually reviewed config options with my bare eyes. Think I'll just need to compose a subscriber down the line. fixed in #c8b17c2 |
Nice, thanks! 🥳 |
Hey, thanks for creating this fork and maintaining it. I was wondering how you'd feel about also merging this PR in to avoid the
origin
field with filename/line number being added? There's some justification in the original issue.(Maybe there's a more nuanced solution like allowing the user to filter the fields somehow, not sure what you'd prefer!)
The text was updated successfully, but these errors were encountered: