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

Idea: Linting support for docker-compose.yml files #84

Closed
lostintangent opened this issue Oct 28, 2016 · 14 comments
Closed

Idea: Linting support for docker-compose.yml files #84

lostintangent opened this issue Oct 28, 2016 · 14 comments
Labels
enhancement New feature or request wontfix This will not be worked on

Comments

@lostintangent
Copy link
Member

It would make it a lot simpler to author Dockerfile and docker-compose.yml files if you could view actionable linting errors from within VS Code. For example, Docker Compose includes JSON schema files for its various formats (e.g. v2.0) that document how to determine whether a compose file is valid or not.

@lostintangent lostintangent changed the title [Feature Request] Linting support for Dockerfile and docker-compose.yml files Linting support for Dockerfile and docker-compose.yml files Nov 28, 2016
@lostintangent
Copy link
Member Author

Updating this to mention just docker-compose.yml files, since we recently introduced linting suport for Dockerfiles.

@lostintangent lostintangent changed the title Linting support for Dockerfile and docker-compose.yml files Linting support for docker-compose.yml files Dec 5, 2016
StephenWeatherford referenced this issue in microsoft/vscode-docker Aug 27, 2018
* Added Azure Credentials Manager Singleton (#18)

* Added Azure Credentials Manager Singleton
* Added getResourceManagementClient

* Sorted Existing Create Registry ready for code review

* Added acquiring telemetry data for create registry

* broke up createnewresourcegroup method and fixed client use

Added try catch loop and awaited for resource group list again to check for duplicates with ResourceManagementClient

* Jackson esteban/unified client nit Fix (#24)

* Added Azure Credentials Manager Singleton

* Small Style Fixes

* Further Style fixes, added getResourceManagementClient

* Lazy Initialization Patches

* Enabled location selection

* Location request fixes

-Changed order of questions asked to user for better UX (location of new res group & location of new registry)
-Placeholder of location is display name view

* Refactor while loop for new res group

* Added SKU selection

* Quick fix- initializing array syntax

* Added specific error messages and comments

* Julia/delete image (#29)

* first fully functional version of delete through input bar AND right click

* refactored code to make it prettier!

* comments

* comments, added subscription function

* fixed to style guide

* style fixes, refactoring

* delete image after reviews

put my functions from azureCredentialsManager into two new files: utils/azure/acrTools.ts, and commands/utils/quick-pick-azure.ts

Edited code based on Esteban's and Bin's reviews

* One last little change to delete image

* moved repository, azureimage, and getsubscriptions to the correct places within deleteImage

* changes from PR reviews on delete image

* fixed authentication issue, got rid of azureAccount property for repository and image

**on constructor for repository, azurecredentialsmanager was being recreated and thus couldn't find the azureAccount. For this reason, I got rid of the azureAccount property of the classes Repository and AzureImage. This bug may lead to future problems (Esteban and I couldn't see why it was happening)

* minor fixes deleteImage

* delete a parentheses

* Estebanreyl/dev merge fixes  (#43)

* Merge fixes to acquire latest telemetry items

* Updated to master AzureUtilityManager

* Rutusamai/list build tasks for each registry (#37)

* tslint updates, transfered from old branch

* updated icon

* Addressed PR comments- mostly styling/code efficiency

* Changed url to aka.ms link

* changed Error window to Info message for no build tasks in your registry

* Changed default sku and unified parsing resource group into a new method, getResourceGroup in acrTools.ts

* Changed build task icon

* Julia/delete repository final (#49)

* deleteRepo moved over to branch off dev

* Got rid of unnecessary code, fully functioning!

* deleteRepo moved over to branch off dev

* Got rid of unnecessary code, fully functioning!

* spacing

* final commit

* Cleaned code

* Added Telemetry

* Julia/delete registry final (#47)


Delete azure registry functionality added

Delete azure registry moved to branch off dev

Reorganized stye

* began updating

* Reorganized create registry and delete azure image

* continued improvements

* Began updating login

* being credentials update

* further updates

* Finished updating, need to test functionality now

* Updated requests, things all work now

* Applied some nit fixes

* Updates to naming

* maintain UtilityManager standards

* Updated Prompts

* Updated imports and naming / standarized telemetry

* Added explorer refresh capabilities on delete/add

* Remove build task features from this branch

This reverts commit 126c01e.

* Merge bugfixes and name specification

* updated weird naming issue

* Deleted deprecated telemetry, added copyright comment and updated quick picks

* Updated casing

* Updated resource group and registry validity checking and other nit fixes

* Updated Azure Utility Manager to by default sort registries alphabetically

* Updated azureRegistryNodes and registryRootNode to use shared functions

* Corrected resourcegroup name test

* added delete button when deleting an image

* Small changes in variables for better prompts and success notifications
StephenWeatherford referenced this issue in microsoft/vscode-docker Nov 9, 2018
* Added Azure Credentials Manager Singleton (#18)

* Added Azure Credentials Manager Singleton
* Added getResourceManagementClient

* Sorted Existing Create Registry ready for code review

* Added acquiring telemetry data for create registry

* broke up createnewresourcegroup method and fixed client use

Added try catch loop and awaited for resource group list again to check for duplicates with ResourceManagementClient

* Jackson esteban/unified client nit Fix (#24)

* Added Azure Credentials Manager Singleton

* Small Style Fixes

* Further Style fixes, added getResourceManagementClient

* Lazy Initialization Patches

* Enabled location selection

* Location request fixes

-Changed order of questions asked to user for better UX (location of new res group & location of new registry)
-Placeholder of location is display name view

* Refactor while loop for new res group

* Added SKU selection

* Quick fix- initializing array syntax

* Added specific error messages and comments

* Julia/delete image (#29)

* first fully functional version of delete through input bar AND right click

* refactored code to make it prettier!

* comments

* comments, added subscription function

* fixed to style guide

* style fixes, refactoring

* delete image after reviews

put my functions from azureCredentialsManager into two new files: utils/azure/acrTools.ts, and commands/utils/quick-pick-azure.ts

Edited code based on Esteban's and Bin's reviews

* One last little change to delete image

* moved repository, azureimage, and getsubscriptions to the correct places within deleteImage

* changes from PR reviews on delete image

* fixed authentication issue, got rid of azureAccount property for repository and image

**on constructor for repository, azurecredentialsmanager was being recreated and thus couldn't find the azureAccount. For this reason, I got rid of the azureAccount property of the classes Repository and AzureImage. This bug may lead to future problems (Esteban and I couldn't see why it was happening)

* minor fixes deleteImage

* delete a parentheses

* copied previous push to acr into new pull-from-azure.ts file

* Estebanreyl/dev merge fixes  (#43)

* Merge fixes to acquire latest telemetry items

* Updated to master AzureUtilityManager

* added acrbuild stuff

* added acrbuild stuff

* Update to match utility manager class

* Rutusamai/list build tasks for each registry (#37)

* tslint updates, transfered from old branch

* updated icon

* Addressed PR comments- mostly styling/code efficiency

* Changed url to aka.ms link

* changed Error window to Info message for no build tasks in your registry

* Changed default sku and unified parsing resource group into a new method, getResourceGroup in acrTools.ts

* Changed build task icon

* Added quick pick for selecting resource group and registry

* clean

* Added subscription support

* utility bug fix

* Julia/delete repository final (#49)

* deleteRepo moved over to branch off dev

* Got rid of unnecessary code, fully functioning!

* deleteRepo moved over to branch off dev

* Got rid of unnecessary code, fully functioning!

* spacing

* final commit

* Cleaned code

* Added Telemetry

* Julia/delete registry final (#47)


Delete azure registry functionality added

Delete azure registry moved to branch off dev

Reorganized stye

* Made loginCredentials method in acrTools more efficient, added Pull Image from Azure option, temporary fix for no registries

* added folder select

* Split the loginCredentials function, added docker login and docker pull and telemetry actions

* Finished pull from azure right click feature

* deleted push to azure

* removed username and password from Azure Registry Node

* Clean up

* copied previous push to acr into new pull-from-azure.ts file

* Made loginCredentials method in acrTools more efficient, added Pull Image from Azure option, temporary fix for no registries

* Split the loginCredentials function, added docker login and docker pull and telemetry actions

* Finished pull from azure right click feature

* deleted push to azure

* Clean up

* Working again after rebasing and resolving merge conflicts

* Updates and fixes

* Fixes from PR comments

-renamed loginCredentials functions
-added await to docker commands in image pull
-cleanup

* uncapitalize AzureRegistryNode

* Refactoring, prod.

* Flexible OSType

* Estebanreyl/ready for production (#55)

* began updating

* Reorganized create registry and delete azure image

* continued improvements

* Began updating login

* being credentials update

* further updates

* Finished updating, need to test functionality now

* Updated requests, things all work now

* Applied some nit fixes

* Updates to naming

* maintain UtilityManager standards

* Updated Prompts

* Updated imports and naming / standarized telemetry

* Added explorer refresh capabilities on delete/add

* Jackson/quick build dev (#46)

* added acrbuild stuff

* added acrbuild stuff

* Update to match utility manager class

* Added quick pick for selecting resource group and registry

* clean

* Added subscription support

* utility bug fix

* added folder select

* Updates and fixes

* Refactoring, prod.

* Flexible OSType

* added ID

* Move build to azure commands

* cleanup

* Relative dockerfile support

* Removed await, updating list in enumeration

* fixed chdir

* flexible ostype

* Rutusamai/Show Build Task properties (#70)

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show is working- final

* removed run build task stuff

* cleanup

* addressed esteban's comments

* buildTask = context.task.name rather than label

* fixes from Bin's comments

* Merge branch 'dev' of https://github.com/AzureCR/vscode-docker into master4

* merge

* missed small changes

* Rutusamai/Run a Build Task (#71)

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show is working- final

* run is working for right click

* run build task works through command pallette. added success notification

* removed show build task

* cleanup- matched quickpick buils task and tasknode withshow build task branch

* removed showTaskManager

* spacing

* outputChannel and null icon

* merge to update with dev

* Estebanreyl/build logs final (#72)

*  "Merge branch 'estebanreyl/buildlogsWithAccesibility' of https://github.com/AzureCR/vscode-docker into estebanreyl/buildlogsWithAccesibility"

This reverts commit e645cbf, reversing
changes made to fc4a477.

* Refactored and organized log view into readable components.

* moved storage

* Began incorporating icons

* Added icons

* Further standarization

* added logs

* Added download log capabilities

* Updated Copy

* Updated inner div size alignment

* Added resizing script

* header sort by is only clickable on text now

* fixed header table

* Fix minor display issues

* Identified filtering strategy

* Begin adding sorting

* Merge with dev

* Added proper filtering

* Nit loading improvements

* Added accesibility, key only behaviour

* accesibility retouches and enable right click from tasknode

* merges

* fix module name

* Adds streaming and command standarization (ext.ui) (#73)

* Adds streaming and command standarization (ext.ui)

* removed unecessary append lines

* small fixes

* Fix merge issues

* changes for ACR 3.0.0 (#80)

* This commit contains changes necessary for the
azure-arm-containerregistry 3.0.0

* Fixed PR feedback

* Run task fixed. Issue ID: 79

* missing changes added

* Fixing ACR run logs for Images

* Sajaya/top1 (#83)

* Query only 1 record for runs

* View Azure logs

* Refactoring build to run and buildTask to task

* Update Credentail Management

Sorted Existing Create Registry ready for code review

Jackson esteban/unified client nit Fix (#24)

* Added Azure Credentials Manager Singleton

* Small Style Fixes

* Further Style fixes, added getResourceManagementClient

* Lazy Initialization Patches

Enabled location selection

Location request fixes

-Changed order of questions asked to user for better UX (location of new res group & location of new registry)
-Placeholder of location is display name view

Added SKU selection

Quick fix- initializing array syntax

Julia/delete image (#29)

* first fully functional version of delete through input bar AND right click

* refactored code to make it prettier!

* comments

* comments, added subscription function

* fixed to style guide

* style fixes, refactoring

* delete image after reviews

put my functions from azureCredentialsManager into two new files: utils/azure/acrTools.ts, and commands/utils/quick-pick-azure.ts

Edited code based on Esteban's and Bin's reviews

* One last little change to delete image

* moved repository, azureimage, and getsubscriptions to the correct places within deleteImage

* changes from PR reviews on delete image

* fixed authentication issue, got rid of azureAccount property for repository and image

**on constructor for repository, azurecredentialsmanager was being recreated and thus couldn't find the azureAccount. For this reason, I got rid of the azureAccount property of the classes Repository and AzureImage. This bug may lead to future problems (Esteban and I couldn't see why it was happening)

* minor fixes deleteImage

* delete a parentheses

copied previous push to acr into new pull-from-azure.ts file

Estebanreyl/dev merge fixes  (#43)

* Merge fixes to acquire latest telemetry items

* Updated to master AzureUtilityManager

added acrbuild stuff

Rutusamai/list build tasks for each registry (#37)

* tslint updates, transfered from old branch

* updated icon

* Addressed PR comments- mostly styling/code efficiency

* Changed url to aka.ms link

* changed Error window to Info message for no build tasks in your registry

* Changed default sku and unified parsing resource group into a new method, getResourceGroup in acrTools.ts

* Changed build task icon

utility bug fix

Julia/delete repository final (#49)

* deleteRepo moved over to branch off dev

* Got rid of unnecessary code, fully functioning!

* deleteRepo moved over to branch off dev

* Got rid of unnecessary code, fully functioning!

* spacing

* final commit

* Cleaned code

* Added Telemetry

Julia/delete registry final (#47)

Delete azure registry functionality added

Delete azure registry moved to branch off dev

Reorganized stye

Made loginCredentials method in acrTools more efficient, added Pull Image from Azure option, temporary fix for no registries

Split the loginCredentials function, added docker login and docker pull and telemetry actions

copied previous push to acr into new pull-from-azure.ts file

Made loginCredentials method in acrTools more efficient, added Pull Image from Azure option, temporary fix for no registries

Split the loginCredentials function, added docker login and docker pull and telemetry actions

Finished pull from azure right click feature

Working again after rebasing and resolving merge conflicts

Refactoring, prod.

Estebanreyl/ready for production (#55)

* began updating

* Reorganized create registry and delete azure image

* continued improvements

* Began updating login

* being credentials update

* further updates

* Finished updating, need to test functionality now

* Updated requests, things all work now

* Applied some nit fixes

* Updates to naming

* maintain UtilityManager standards

* Updated Prompts

* Updated imports and naming / standarized telemetry

* Added explorer refresh capabilities on delete/add

Jackson/quick build dev (#46)

* added acrbuild stuff

* added acrbuild stuff

* Update to match utility manager class

* Added quick pick for selecting resource group and registry

* clean

* Added subscription support

* utility bug fix

* added folder select

* Updates and fixes

* Refactoring, prod.

* Flexible OSType

added ID

Move build to azure commands

cleanup

Relative dockerfile support

Removed await, updating list in enumeration

fixed chdir

flexible ostype

Rutusamai/Show Build Task properties (#70)

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show is working- final

* removed run build task stuff

* cleanup

* addressed esteban's comments

* buildTask = context.task.name rather than label

* fixes from Bin's comments

Merge branch 'dev' of https://github.com/AzureCR/vscode-docker into master4

merge

missed small changes

Rutusamai/Run a Build Task (#71)

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show is working- final

* run is working for right click

* run build task works through command pallette. added success notification

* removed show build task

* cleanup- matched quickpick buils task and tasknode withshow build task branch

* removed showTaskManager

* spacing

* outputChannel and null icon

* merge to update with dev

Estebanreyl/build logs final (#72)

*  "Merge branch 'estebanreyl/buildlogsWithAccesibility' of https://github.com/AzureCR/vscode-docker into estebanreyl/buildlogsWithAccesibility"

This reverts commit e645cbf, reversing
changes made to fc4a477.

* Refactored and organized log view into readable components.

* moved storage

* Began incorporating icons

* Added icons

* Further standarization

* added logs

* Added download log capabilities

* Updated Copy

* Updated inner div size alignment

* Added resizing script

* header sort by is only clickable on text now

* fixed header table

* Fix minor display issues

* Identified filtering strategy

* Begin adding sorting

* Merge with dev

* Added proper filtering

* Nit loading improvements

* Added accesibility, key only behaviour

* accesibility retouches and enable right click from tasknode

* merges

* fix module name

Adds streaming and command standarization (ext.ui) (#73)

* Adds streaming and command standarization (ext.ui)

* removed unecessary append lines

* small fixes

* Fix merge issues

changes for ACR 3.0.0 (#80)

* This commit contains changes necessary for the
azure-arm-containerregistry 3.0.0

* Fixed PR feedback

Run task fixed. Issue ID: 79

missing changes added

Fixing ACR run logs for Images

Sajaya/top1 (#83)

* Query only 1 record for runs

* View Azure logs

bug fix

* Removed filter for top (#88)

* fixing Image Log Filter

* fixing tslint error messages

* tslint fixes 2

* First PR #506 review Update include:
	Deletion of resizable.js
	Package.json commands alphabetically ordered.
	General typos, grammar and naming fixes.
	Change from fs - fs-extra.
	Other general improvments.

* Hide Azure Quick Build if Azure Account not available

* Second PR #506 review update.
	Quick Build Image name and dockerFile selection Improvements.
	Upgrade from fs to fs-extra.
	Log table bug fixes.
	Error Handeling Improvements.
	Other general improvements.

* Improving Logs generation Error handeling

* Third PR #506 review update.
	loadLogs parameters update.
	uploadSourceCode Improvements.

* UploadSourceCode no longer has to change process working directory.

* lint fix
StephenWeatherford referenced this issue in microsoft/vscode-docker Nov 17, 2018
* Added Azure Credentials Manager Singleton (#18)

* Added Azure Credentials Manager Singleton
* Added getResourceManagementClient

* Sorted Existing Create Registry ready for code review

* Added acquiring telemetry data for create registry

* broke up createnewresourcegroup method and fixed client use

Added try catch loop and awaited for resource group list again to check for duplicates with ResourceManagementClient

* Jackson esteban/unified client nit Fix (#24)

* Added Azure Credentials Manager Singleton

* Small Style Fixes

* Further Style fixes, added getResourceManagementClient

* Lazy Initialization Patches

* Enabled location selection

* Location request fixes

-Changed order of questions asked to user for better UX (location of new res group & location of new registry)
-Placeholder of location is display name view

* Refactor while loop for new res group

* Added SKU selection

* Quick fix- initializing array syntax

* Added specific error messages and comments

* Julia/delete image (#29)

* first fully functional version of delete through input bar AND right click

* refactored code to make it prettier!

* comments

* comments, added subscription function

* fixed to style guide

* style fixes, refactoring

* delete image after reviews

put my functions from azureCredentialsManager into two new files: utils/azure/acrTools.ts, and commands/utils/quick-pick-azure.ts

Edited code based on Esteban's and Bin's reviews

* One last little change to delete image

* moved repository, azureimage, and getsubscriptions to the correct places within deleteImage

* changes from PR reviews on delete image

* fixed authentication issue, got rid of azureAccount property for repository and image

**on constructor for repository, azurecredentialsmanager was being recreated and thus couldn't find the azureAccount. For this reason, I got rid of the azureAccount property of the classes Repository and AzureImage. This bug may lead to future problems (Esteban and I couldn't see why it was happening)

* minor fixes deleteImage

* delete a parentheses

* copied previous push to acr into new pull-from-azure.ts file

* Estebanreyl/dev merge fixes  (#43)

* Merge fixes to acquire latest telemetry items

* Updated to master AzureUtilityManager

* added acrbuild stuff

* added acrbuild stuff

* Update to match utility manager class

* Rutusamai/list build tasks for each registry (#37)

* tslint updates, transfered from old branch

* updated icon

* Addressed PR comments- mostly styling/code efficiency

* Changed url to aka.ms link

* changed Error window to Info message for no build tasks in your registry

* Changed default sku and unified parsing resource group into a new method, getResourceGroup in acrTools.ts

* Changed build task icon

* Added quick pick for selecting resource group and registry

* clean

* Added subscription support

* utility bug fix

* Julia/delete repository final (#49)

* deleteRepo moved over to branch off dev

* Got rid of unnecessary code, fully functioning!

* deleteRepo moved over to branch off dev

* Got rid of unnecessary code, fully functioning!

* spacing

* final commit

* Cleaned code

* Added Telemetry

* Julia/delete registry final (#47)


Delete azure registry functionality added

Delete azure registry moved to branch off dev

Reorganized stye

* Made loginCredentials method in acrTools more efficient, added Pull Image from Azure option, temporary fix for no registries

* added folder select

* Split the loginCredentials function, added docker login and docker pull and telemetry actions

* Finished pull from azure right click feature

* deleted push to azure

* removed username and password from Azure Registry Node

* Clean up

* copied previous push to acr into new pull-from-azure.ts file

* Made loginCredentials method in acrTools more efficient, added Pull Image from Azure option, temporary fix for no registries

* Split the loginCredentials function, added docker login and docker pull and telemetry actions

* Finished pull from azure right click feature

* deleted push to azure

* Clean up

* Working again after rebasing and resolving merge conflicts

* Updates and fixes

* Fixes from PR comments

-renamed loginCredentials functions
-added await to docker commands in image pull
-cleanup

* uncapitalize AzureRegistryNode

* Refactoring, prod.

* Flexible OSType

* Estebanreyl/ready for production (#55)

* began updating

* Reorganized create registry and delete azure image

* continued improvements

* Began updating login

* being credentials update

* further updates

* Finished updating, need to test functionality now

* Updated requests, things all work now

* Applied some nit fixes

* Updates to naming

* maintain UtilityManager standards

* Updated Prompts

* Updated imports and naming / standarized telemetry

* Added explorer refresh capabilities on delete/add

* Jackson/quick build dev (#46)

* added acrbuild stuff

* added acrbuild stuff

* Update to match utility manager class

* Added quick pick for selecting resource group and registry

* clean

* Added subscription support

* utility bug fix

* added folder select

* Updates and fixes

* Refactoring, prod.

* Flexible OSType

* added ID

* Move build to azure commands

* cleanup

* Relative dockerfile support

* Removed await, updating list in enumeration

* fixed chdir

* flexible ostype

* Rutusamai/Show Build Task properties (#70)

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show is working- final

* removed run build task stuff

* cleanup

* addressed esteban's comments

* buildTask = context.task.name rather than label

* fixes from Bin's comments

* Merge branch 'dev' of https://github.com/AzureCR/vscode-docker into master4

* merge

* missed small changes

* Rutusamai/Run a Build Task (#71)

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show build task works through input bar

* added run build task

* Right click not working on BuildTaskNode. Works through command palette

* quick fixes

* quick fix to enable context menu on buildTaskNode

* comamnd not sending via right click

* working from right click now. trying to do show in json

* Acquire properties in JSON format internally, works

* Now shows organized task jsons on right click

* import

* to do

* issues with getImagesByRepository

* try catch build step

* acrTools

* small refactor

* Show is working- final

* run is working for right click

* run build task works through command pallette. added success notification

* removed show build task

* cleanup- matched quickpick buils task and tasknode withshow build task branch

* removed showTaskManager

* spacing

* outputChannel and null icon

* merge to update with dev

* Estebanreyl/build logs final (#72)

*  "Merge branch 'estebanreyl/buildlogsWithAccesibility' of https://github.com/AzureCR/vscode-docker into estebanreyl/buildlogsWithAccesibility"

This reverts commit e645cbf, reversing
changes made to fc4a477.

* Refactored and organized log view into readable components.

* moved storage

* Began incorporating icons

* Added icons

* Further standarization

* added logs

* Added download log capabilities

* Updated Copy

* Updated inner div size alignment

* Added resizing script

* header sort by is only clickable on text now

* fixed header table

* Fix minor display issues

* Identified filtering strategy

* Begin adding sorting

* Merge with dev

* Added proper filtering

* Nit loading improvements

* Added accesibility, key only behaviour

* accesibility retouches and enable right click from tasknode

* merges

* fix module name

* Adds streaming and command standarization (ext.ui) (#73)

* Adds streaming and command standarization (ext.ui)

* removed unecessary append lines

* small fixes

* Fix merge issues

* changes for ACR 3.0.0 (#80)

* This commit contains changes necessary for the
azure-arm-containerregistry 3.0.0

* Fixed PR feedback

* Run task fixed. Issue ID: 79

* missing changes added

* Fixing ACR run logs for Images

* Sajaya/top1 (#83)

* Query only 1 record for runs

* View Azure logs

* Refactoring build to run and buildTask to task

* Removed filter for top (#88)

* adding run yaml file

* Refactoring to run task file.

* fixing logs filter for images

* Last Update time Fixed

* Cleanup + refactoring delete image to untag image

* Adding delete ACR Image (delete digest)

* Changing text promt on right click to run ACR task file

* Update settings.json

* minor PR review fixes 1

* PR fixes 1

* Missed: change any to string

* merge clean up

* Schedule run code reduction + minor improvements

* ACR request Improvements

* Minor grammar fixes
@ssbarnea
Copy link

ssbarnea commented Dec 8, 2018

Any chance to see this soon?

@PrashanthCorp
Copy link

@rcjsuen is supporting this feature on your roadmap?

@rcjsuen
Copy link

rcjsuen commented Mar 7, 2019

@PrashanthCorp I have considered this in the past but never even got started working on a prototype.

I assumed people would use some sort of YAML editor/validation tool to support this use case...but maybe not everyone wants to install a YAML editor.

@SuperSandro2000
Copy link

A docker-compose.yml file can be valid yaml but not valid for docker-compose. I think linting if keys exist and have proper values set would be very valuable for this extension.

@rcjsuen
Copy link

rcjsuen commented Jul 6, 2019

Anyone on this ticket that needs this should be able to get something decent with Red Hat's YAML extension. You can use something like the following in your settings.json file.

{
    "yaml.schemas": {
        "https://raw.githubusercontent.com/docker/cli/master/cli/compose/schema/data/config_schema_v3.8.json": "docker-compose.yml"
    }
}

@SuperSandro2000
Copy link

A complete list of all versions is available here https://github.com/docker/compose/tree/master/compose/config

@bwateratmsft
Copy link
Contributor

Thanks for finding that @SuperSandro2000!

@JustinGrote
Copy link

This is also available on schemastore:
https://www.schemastore.org/json/

@bwateratmsft
Copy link
Contributor

Duping to microsoft/vscode-docker#2539

@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 28, 2021
@bwateratmsft bwateratmsft reopened this Jan 26, 2022
@microsoft microsoft unlocked this conversation Jan 26, 2022
@bwateratmsft bwateratmsft transferred this issue from microsoft/vscode-docker Jan 26, 2022
@bwateratmsft bwateratmsft changed the title Linting support for docker-compose.yml files Idea: Linting support for docker-compose.yml files Jan 26, 2022
@bwateratmsft bwateratmsft added the enhancement New feature or request label Jan 26, 2022
@ucheNkadiCode ucheNkadiCode added investigate Issue needs investigation P1 labels Jan 31, 2022
@bwateratmsft bwateratmsft self-assigned this Jan 31, 2022
@bwateratmsft bwateratmsft removed investigate Issue needs investigation P1 labels May 10, 2023
@bwateratmsft bwateratmsft removed their assignment May 10, 2023
@bwateratmsft
Copy link
Contributor

bwateratmsft commented May 10, 2023

I've investigated this and ultimately the conclusion I've reached is that without just straight-up copying the functionality of the YAML extension, I could not do as good of a job as they've already done. So rather than try to implement this, I'm going to close this issue, since with #122 it is now much more user-friendly to have both the YAML and Docker extensions' language services for compose files working side-by-side.

Orthogonally, the compose spec is short of a lot of features (e.g. descriptions, compose-spec/compose-spec#138) and could use some attention, but that would be necessary even if we added schema validation to this language service.

@bwateratmsft bwateratmsft closed this as not planned Won't fix, can't repro, duplicate, stale May 10, 2023
@bwateratmsft bwateratmsft added the wontfix This will not be worked on label May 10, 2023
@AshutoshSajan
Copy link

Anyone on this ticket that needs this should be able to get something decent with Red Hat's YAML extension. You can use something like the following in your settings.json file.

{
"yaml.schemas": {
"https://raw.githubusercontent.com/docker/cli/master/cli/compose/schema/data/config_schema_v3.8.json": "docker-compose.yml"
}
}

there is no such file on GitHub so which file we should use now? or does any other extension support it?

@bwateratmsft
Copy link
Contributor

I'm still able to access that file, what error do you see?

@AshutoshSajan
Copy link

AshutoshSajan commented Aug 26, 2023

I am getting the following error
Docker compose version is v2.20..2

Unable to load schema from 'https://raw.githubusercontent.com/docker/cli/master/cli/compose/schema/data/config_schema_v3.8.json': Request vscode/content failed unexpectedly without providing any details.YAML(768)
The version of the Docker Compose document

No quick fixes available

is this docker compose version issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

9 participants