New Feature: Web parser with pattern for Piers #957
Conversation
- Update zlib source to version 1.3.1 - Modify CMake configuration to support static library build - Disable example binaries - Update SHA256 checksum
alpgul
changed the title
|
Thanks for all the updates, can you merge the last 3 commits and I'll do a full review in the coming days? |
|
Once this passes review the last step will be to add a changelog entry and mini version increment (the middle number as its a new feature) in |
alpgul
force-pushed
the
Piers-webParserWithPattern
branch
from
f6d9812 to
22486fe
Compare
There was a problem hiding this comment.
Just some minors but otherwise looks good.
Please remove the windows build script, I must check on whether or not that can go in but no need to hold up this PR.
Outstanding things that need to be done:
- Changelog entry
- Minor version increment in addon.xml.in
- Update to README.md to explain the feature, otherwise users won't know it's there. At least some additions to the examples are required.
alpgul
force-pushed
the
Piers-webParserWithPattern
branch
from
22486fe to
3fbdbeb
Compare
This reverts commit 056c53b.
alpgul
force-pushed
the
Piers-webParserWithPattern
branch
from
3fbdbeb to
d52395e
Compare
|
Code formatting issues have been fixed. The changelog has been updated, the minor version in addon.xml.in has been incremented, and the README.md has been updated with explanations and examples of the new feature. The Windows build script has also been removed. |
- Add WebStreamExtractor class to handle web stream URL extraction - Improve stream URL parsing in StreamUtils - Add CurlUtils for HTTP request handling - Update WebUtils with improved web request functionality - Update documentation and changelog
alpgul
force-pushed
the
Piers-webParserWithPattern
branch
from
d52395e to
463eae8
Compare
Adapts the "Web Parser with Regex #950 " feature, originally developed for the Omega version, to the Piers version.