Skip to content

Add spaces to match_groups#5

Open
KevinColemanInc wants to merge 2 commits into
ofeldt:masterfrom
KevinColemanInc:master
Open

Add spaces to match_groups#5
KevinColemanInc wants to merge 2 commits into
ofeldt:masterfrom
KevinColemanInc:master

Conversation

@KevinColemanInc

Copy link
Copy Markdown
Contributor

No description provided.

@FreedomBen

Copy link
Copy Markdown

@KevinColemanInc I noticed this PR has been open a really long time. Did you ever hear back? I wonder if this project has been abandoned.

@KevinColemanInc

Copy link
Copy Markdown
Contributor Author

I ended up just forking and writing my own :-/. The DIY approach let me customize the interface to better fit my business needs.

I assume this is abandoned. RIP.

@FreedomBen

Copy link
Copy Markdown

Ah cool, did you open your own up?

@KevinColemanInc

Copy link
Copy Markdown
Contributor Author

Actually, it looks like the author accepted my other change in late-2019:

315bcfc

Maybe they just didn't like this change.

@KevinColemanInc

Copy link
Copy Markdown
Contributor Author

My version was tied to the application's business logic. We were using regexes to capture information from pdf documents and websites. I added some nice features pre-loading the sample text and regex fields with the html and extracted pdf text, rendering the documents in an iframe, and opening up the regex groups into a new window for further parsing :)

@KevinColemanInc

Copy link
Copy Markdown
Contributor Author

If you need something quick, you could use https://rubular.com/

We had to self-host, because rubular changed the ruby version and broke all of our regexes that worked on 2.4 :-/

@FreedomBen

Copy link
Copy Markdown

Oh haha, nice. Yeah I've been using Rubular for years but it sends a POST back to it's server for each regex check. Most of the time I don't care but I'm working on some sensitive code right now that I don't want sent to a closed-source backend. I just dockerized Rubulex and deployed it for myself. I offered the code back up, I guess we'll see if the maintainer wants it 🤷‍♂️

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