-
Notifications
You must be signed in to change notification settings - Fork 166
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
Add azd intellij plugin into azure-toolkit-for-intellij #9712
base: develop
Are you sure you want to change the base?
Add azd intellij plugin into azure-toolkit-for-intellij #9712
Conversation
enable azd init for pom.xml, azd up for azure.yaml
public void update(@NotNull AnActionEvent event) { | ||
VirtualFile file = event.getData(CommonDataKeys.VIRTUAL_FILE); | ||
// Only enable the action for specific files | ||
boolean enabled = file != null && getSupportedFiles().contains(file.getName()); |
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 enable the azd
related actions for only specific file patterns (such as pom.xml
, azure.yaml
). For other files or directories, the user should not see azd
actions when clicking azure
.
Messages.getQuestionIcon() | ||
); | ||
if (result == Messages.YES) { | ||
if (!AzdCliUtils.installAzdCli(project)) { |
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.
Install successfully first, then run azd
commands in the terminal.
private static AzdVersion cachedAzdVersion = null; | ||
|
||
@Nullable | ||
public static AzdVersion getAzdVersion() { |
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 method is currently used to check if azd
cli is installed locally.
return null; | ||
} | ||
|
||
public static class AzdVersion { |
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.
commandLine.withExePath("powershell") | ||
.withParameters("-ex", "AllSigned") | ||
.withParameters("-c", "Invoke-RestMethod 'https://aka.ms/install-azd.ps1' | Invoke-Expression"); | ||
} else if (SystemInfo.isLinux || SystemInfo.isMac) { |
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.
Same command for Linux and Mac platforms.
@@ -165,6 +165,11 @@ public RedisAccessKeys regenerateKey(RedisKeyType keyType) { | |||
return new RedisAccessKeysMock(); | |||
} | |||
|
|||
@Override |
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.
Fix the error when mvn clean install -f Utils/pom.xml
in the CONTRIBUTING.md.
What does this implement/fix? Explain your changes.
This PR is to add
azd intellij plugin
intoazure-toolkit-for-intellij
.azd
cli when clickingazd intellij plugin
.azd
installed and prompt user to install.azd init
forpom.xml
, andazd up
forazure.yaml
.azure-toolkit-ide-hdinsight-libs
.azd
is an open-source tool that accelerates provisioning and deploying app resources on Azure, see here for more details.Does this close any currently open issues?
No
Any relevant logs, screenshots, error output, etc.?
Any other comments?
No
Has this been tested?