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

DX-1669: Add ide:info command #345

Merged
merged 3 commits into from
Nov 6, 2020
Merged

Conversation

danepowell
Copy link
Contributor

@anavarre before I add tests, how do you feel about this?

@danepowell
Copy link
Contributor Author

The test failure is unrelated, I'll research and fix it separately.

@danepowell
Copy link
Contributor Author

Tests fixed, it seems Chocolatey is having reliability issues: chocolatey/choco#1629

@anavarre
Copy link
Contributor

anavarre commented Nov 5, 2020

Tried to checkout the PR to test the new command but I'm not seeing it yet. Let me know when it's okay to test.

@danepowell
Copy link
Contributor Author

@anavarre commands are cached, did you clear the kernel cache? acli ckc

@anavarre
Copy link
Contributor

anavarre commented Nov 5, 2020

Didn't know about acli ckc. That helped, thanks.

Manual testing:

$ ./bin/acli ide:info
Please select a Cloud Platform application:
  [0] mytestapp
 > 0
Please select an IDE to get more information:
  [0] mytestide
 > 0
 ------------------- -------------------------------------------------------------------------------- 
  IDE property        IDE value                                                                       
 ------------------- -------------------------------------------------------------------------------- 
  UUID                <REDACTED>                                            
  Label               test                                                                            
  Owner name          Aurelien Navarre                                                                
  Owner username      anavarre                                                                        
  Owner email         <REDACTED>                                                     
  Cloud application   https://cloud.acquia.com/api/applications/<REDACTED>  
  IDE URL             https://<REDACTED>.ide.ahdev.cloud                    
  Web URL             https://<REDACTED>.web.ahdev.cloud                    
 ------------------- --------------------------------------------------------------------------------

@danepowell danepowell merged commit 914c7f8 into acquia:master Nov 6, 2020
@danepowell danepowell deleted the DX-1669 branch September 22, 2021 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants