Skip to content
This repository has been archived by the owner on Nov 20, 2023. It is now read-only.

#1158 Feature/dashboard port in tye yaml #1163

Merged

Conversation

daniel-simpson
Copy link
Contributor

Feature implementation for #1158

It is a relatively small, and self-contained change with tests added, but I understand if it isn't something you want implemented in this project! Submitting this PR to further discussion in the #1158 issue.

Thank again for everything, I've reaally enjoyed using Tye over the past six months or so.

@dnfadmin
Copy link

dnfadmin commented Sep 4, 2021

CLA assistant check
All CLA requirements met.

Copy link
Contributor

@philliphoff philliphoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, with just one comment re: the configuration file schema. The additional tests are very much appreciated, too.

@daniel-simpson
Copy link
Contributor Author

I've added the requested change into the schema file.

Thanks again for the great project :D

@philliphoff philliphoff merged commit 2858fce into dotnet:main Sep 30, 2021
@daniel-simpson daniel-simpson deleted the feature/dashboard-port-in-tye-yaml branch September 30, 2021 08:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants