Skip to content

Conversation

@tony
Copy link
Member

@tony tony commented Jul 10, 2022

Parser:

  • Git scp-style URLs: Fix detection detect of scp style URLs with .git prefixes
  • Fix critical bug where matchers were being applied as variable not annotations
  • Fix pattern_defaults
  • Base tests: Rename Location to URL
  • Tests and modules: Rename _location to _url

@codecov
Copy link

codecov bot commented Jul 10, 2022

Codecov Report

Merging #386 (efe929f) into master (0f2d30b) will increase coverage by 0.28%.
The diff coverage is 90.69%.

@@            Coverage Diff             @@
##           master     #386      +/-   ##
==========================================
+ Coverage   73.34%   73.63%   +0.28%     
==========================================
  Files          26       26              
  Lines        2165     2185      +20     
==========================================
+ Hits         1588     1609      +21     
+ Misses        577      576       -1     
Impacted Files Coverage Δ
libvcs/parse/svn.py 93.87% <60.00%> (-1.96%) ⬇️
libvcs/parse/hg.py 95.91% <66.66%> (-2.00%) ⬇️
libvcs/parse/base.py 88.23% <100.00%> (+6.98%) ⬆️
libvcs/parse/git.py 100.00% <100.00%> (+1.58%) ⬆️
tests/parse/test_git.py 100.00% <100.00%> (ø)
tests/parse/test_hg.py 100.00% <100.00%> (ø)
tests/parse/test_svn.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f2d30b...efe929f. Read the comment docs.

tony added a commit that referenced this pull request Jul 10, 2022
@tony tony changed the title Parser: Support explicit matchers Parser: Support explicit matchers, critical bug fixes Jul 10, 2022
@tony tony merged commit e1641fc into master Jul 10, 2022
@tony tony deleted the exact-match branch July 10, 2022 23:32
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.

1 participant