Skip to content

Added validation for repo types as well in vscode and jetbrains #3044

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

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

agrasth
Copy link
Collaborator

@agrasth agrasth commented Jul 29, 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 added safe to test Approve running integration tests on a pull request ignore for release Automatically generated release notes labels 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 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
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