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

Add coverage for containerapps-albumapi-csharp test #706

Open
wants to merge 57 commits into
base: main
Choose a base branch
from

Conversation

motm32
Copy link
Contributor

@motm32 motm32 commented Jun 26, 2024

Fixes #700

Have run this test locally and also using the pipeline and both passed.

@motm32 motm32 requested a review from a team as a code owner June 26, 2024 16:33
@motm32 motm32 changed the title Meganmott/albumapi csharp test Add coverage for containerapps-albumapi-csharp test Jun 26, 2024
@MicroFish91
Copy link
Contributor

MicroFish91 commented Jun 26, 2024

Nice! Could you also add a re-deploy test case as well?

@motm32
Copy link
Contributor Author

motm32 commented Jun 26, 2024

Nice! Could you also add a re-deploy test case as well?

yup!

Base automatically changed from mwf/albumapi-js-test to main June 27, 2024 23:19
@MicroFish91
Copy link
Contributor

Note: Temporary halt on this due to: #697 (comment)

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.

Add coverage for containerapps-albumapi-csharp
2 participants