-
Notifications
You must be signed in to change notification settings - Fork 2
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
NEW (Extension) @W-17330890@ A4D SFCA Integration - Integrate with UnifiedDiff UI #155
Conversation
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - add token to access internal repo
…exCRUDViolation - change imports
…exCRUDViolation - change imports
…ed imports and auths
@@ -33,7 +33,9 @@ | |||
"SFCA" | |||
], | |||
"dependencies": { | |||
"@salesforce/vscode-service-provider": "^1.3.0-rc.6.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We access the LLM model via this package.
"cross-spawn": "^7.0.3", | ||
"einstein-shared": "https://github.com/forcedotcom/salesforcedx-vscode-einstein-shared#semver:0.3.9", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the shared module repository and since the package cannot be open sourced yet, we depend on it this way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When installing our vscode extention… will the new dependency
einstein-shared": "https://github.com/forcedotcom/salesforcedx-vscode-einstein-shared#semver:0.3.9",
get pulled during install or is it installed into the vsix file?
That is... can we even ship this line of code or will it break users during install?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can ship with this line of code. This is bundled into the vsix.
@@ -9,10 +9,14 @@ jobs: | |||
steps: | |||
- name: 'Check out the code' | |||
uses: actions/checkout@v4 | |||
with: | |||
token: ${{ secrets.IDEE_GH_TOKEN }} # So we can access internal repo to pull shared ui components |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This lets us run as idee bot user so we can access the internal private repo.
No description provided.