Skip to content

Conversation

@tony
Copy link
Member

@tony tony commented Sep 23, 2022

Required by vcs-python/vcspull#373

and i dont have time to do this until the weekend

Todo:

  • copy over all possible URL combinations that can happen in .vcspull.yaml configurations
    • add them to a massive pytest.parametrize test grid
    • all of them must ✅ pass
    • To be very generous, even try to allow git+ssh://git@github.com:tony/pytest.git (mix of copying from github + a pip url)
  • see how / if remotes should be resolved on libvcs (and if so, which module) or in vcspull

@codecov
Copy link

codecov bot commented Sep 24, 2022

Codecov Report

Merging #420 (79d9b01) into master (1a14726) will increase coverage by 0.17%.
The diff coverage is 67.32%.

@@            Coverage Diff             @@
##           master     #420      +/-   ##
==========================================
+ Coverage   62.00%   62.17%   +0.17%     
==========================================
  Files          36       39       +3     
  Lines        2811     2911     +100     
  Branches      675      695      +20     
==========================================
+ Hits         1743     1810      +67     
- Misses        640      667      +27     
- Partials      428      434       +6     
Impacted Files Coverage Δ
src/libvcs/_internal/module_loading.py 39.02% <39.02%> (ø)
src/libvcs/_internal/shortcuts.py 75.00% <57.14%> (-11.96%) ⬇️
tests/url/test_registry.py 95.45% <95.45%> (ø)
src/libvcs/url/registry.py 95.83% <95.83%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tony tony force-pushed the vcs-registry branch 5 times, most recently from ad87abd to 158c53f Compare September 24, 2022 21:43
@tony tony changed the title Note to self, vcs-registry: All vcspull compatible URLs must have URLs detected vcs-registry: All vcspull compatible URLs must have URLs detected Sep 24, 2022
@tony tony force-pushed the vcs-registry branch 3 times, most recently from 6c5cda0 to 7e236f8 Compare September 25, 2022 00:37
@tony tony force-pushed the vcs-registry branch 5 times, most recently from 884dc27 to efc7bc6 Compare September 25, 2022 11:11
@tony tony merged commit b0ac185 into master Sep 25, 2022
@tony tony deleted the vcs-registry branch September 25, 2022 13:27
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