Skip to content

Conversation

@agrasth
Copy link
Contributor

@agrasth agrasth commented Jul 24, 2025

  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • The pull request is targeting the dev branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....

Add Repository Type Validation for IDE Integrations (VSCode & JetBrains)

Overview

This PR enhances the IDE integration commands (vscode-config and jetbrains-config) by adding validation to ensure that the specified Artifactory repository is not only present, but also of the correct type (vscode or jetbrains). This prevents misconfiguration and provides clearer error messages to users.

Changes

  • Designed for use in CLI commands and integrations that require strict repository type validation (e.g., IDE integrations in jfrog-cli-artifactory
  • Dependency:
    • Relies on a new utility function (ValidateRepoType) added to jfrog-cli-core.

Related

@agrasth agrasth marked this pull request as draft July 25, 2025 11:05
@agrasth agrasth added the safe to test Approve running integration tests on a pull request label Jul 25, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Jul 25, 2025
@agrasth agrasth added ignore for release Automatically generated release notes safe to test Approve running integration tests on a pull request labels Jul 25, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Jul 28, 2025
@agrasth agrasth added the safe to test Approve running integration tests on a pull request label Jul 29, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Jul 29, 2025
@agrasth agrasth closed this Jul 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore for release Automatically generated release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant