Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom LFS server fix - reworked changes from PR#2706 #3785

Conversation

anatolybolshakov
Copy link
Contributor

@anatolybolshakov anatolybolshakov commented Mar 21, 2022

Added info message for changes from PR.
Related issue - #2598
Tested locally for scenarios with local lfs server:

  • .lfsconfig file does not exist
  • .lfsconfig exists - local lfs server is used to fetch

Tests:

  • Covered cases with .lfsconfig presenting/missing (L0 tests)
  • Refactored ArgUtil static class to make it more testable

@anatolybolshakov anatolybolshakov mentioned this pull request Mar 21, 2022
@anatolybolshakov anatolybolshakov force-pushed the users/anatolybolshakov/fix-custom-lfs-server branch from 58cdd8a to b25fd3d Compare March 22, 2022 20:41
@anatolybolshakov
Copy link
Contributor Author

@mmrazik could you please take a look?

@mmrazik mmrazik force-pushed the users/anatolybolshakov/fix-custom-lfs-server branch from b25fd3d to c31d9df Compare May 9, 2022 14:02
@KonstantinTyukalov
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@alexander-smolyakov
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants