Skip to content
This repository has been archived by the owner on Jun 30, 2022. It is now read-only.

Conversation

matiasroldan6
Copy link
Contributor

@matiasroldan6 matiasroldan6 commented Jan 8, 2021

Related to #3290

Depends on #3505, #3542, #3601, #3732, #3736

Purpose

What is the context of this pull request? Why is it being done?
The Functional Tests pipeline yamls for the C# and TS bots were improved to consume reusable templates.

Changes

Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)
We moved reusable functionality away from the main yamls Nightly-Dotnet-VirtualAssistantToSkill.yml, Nightly-Dotnet-SkillSample.yml, Nightly-TypeScript-VirtualAssistantToSkill.yml, and Nightly-TypeScript-SkillSample.yml and created with that the following templates added in the yamls folder on the root of the repository (which can be changed to another place)

  • cleanupResourcesStep
  • deletePackageLockStep
  • deployStep
  • dotnetBuildStep
  • dotnetLogVersionsStep
  • endLogStep
  • functionalTestsStep
  • nodeBuildStep
  • nodeLogVersionsStep
  • npmInstallRequirementsStep
  • setConfigFileStep
  • setDirectLineStep
  • useNodeStep
  • useNugetStep
  • yeomanInstallationStep
  • yoCreateBotStep

image

Tests

Is this covered by existing tests or new ones? If no, why not?

Successful execution of Nightly-Dotnet-VirtualAssistantToSkill pipeline
image

Successful execution of Nightly-Dotnet-Skill pipeline
image

Successful execution of Nightly-TypeScript-VirtualAssistantToSkill pipeline
image

Successful execution of Nightly-TypeScript-Skill pipeline
image

Feature Plan

Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.

Checklist

General

  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated the appropriate tests
  • I have updated related documentation

@lauren-mills
Copy link
Contributor

Closing - outdated

@Batta32 Batta32 deleted the feature/southworks/functional-tests/apply-improvements-ymls branch June 2, 2021 18:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants