Open kklemon opened 1 week ago
Tag @nking-1
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 59.26%. Comparing base (
ba86754
) to head (dea02aa
).
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The regex for matching urls in the
image
function starts with a$
character (ref), i.e. the "match end of string"-metacharacter, which seems to be wrong. I guess the intention was to match against the start of the string, which could be achieved with a^
character, but sincere.match
matches against the start of a string already, this can also be omitted in this specific case.This led to the situation that urls are generally not recognized and can't be used with the
image
function. This PR attempts to fix this issue by modifying the aforementioned regex and additionally providing tests for the_image
module to prevent such problems in the future.