Skip to content
This repository was archived by the owner on Jan 15, 2025. It is now read-only.

fix luis:build write out version in luis setting when the directVersionPublish is not set#1215

Merged
munozemilio merged 4 commits intomainfrom
shuwan/fixDirectPublish
Apr 26, 2021
Merged

fix luis:build write out version in luis setting when the directVersionPublish is not set#1215
munozemilio merged 4 commits intomainfrom
shuwan/fixDirectPublish

Conversation

@cosmicshuai
Copy link
Contributor

@cosmicshuai cosmicshuai commented Apr 23, 2021

closes: #1215
If --directVersionPublish flag is absent in luis:build command. The version info won't be included in the luis setting file.

@cosmicshuai cosmicshuai changed the title fix luis:build write out version even the directVersionPublish is not set fix luis:build write out version in luis setting when the directVersionPublish is not set Apr 26, 2021
@cosmicshuai cosmicshuai marked this pull request as ready for review April 26, 2021 10:53
@munozemilio munozemilio merged commit 44f6fa3 into main Apr 26, 2021
@munozemilio munozemilio deleted the shuwan/fixDirectPublish branch April 26, 2021 17:32
munozemilio pushed a commit that referenced this pull request Apr 28, 2021
…onPublish is not set (#1215)

* init

* fix tests

* retrigger

* remove redundent ()
munozemilio pushed a commit that referenced this pull request Apr 28, 2021
…onPublish is not set (#1215)

* init

* fix tests

* retrigger

* remove redundent ()
munozemilio added a commit that referenced this pull request Apr 28, 2021
* fix luis cross train unable to reference external files (#1181)

* init

* add path resovle

* fix import path resolve issue

* add tests

* fix lint and unused imports

* modify config name

* retrigger

* refactor

Co-authored-by: Emilio Munoz <emmunozp@microsoft.com>

* fix luis:build write out version in luis setting when the directVersionPublish is not set (#1215)

* init

* fix tests

* retrigger

* remove redundent ()

* up versioned orchestrator-core to 4.13.1 (#1211)

* Update package.json

Use orchestrator-core v 4.13.1

* Update pnpm-lock.yaml

* skip empty lu file, added empty lu file to test

* Update orchestratorhelper.ts

fixed lint issues

* use forward slash for file path in orchestrator.settings.json to be compatible with non windows platform

* addressed PR comments

Co-authored-by: Emilio Munoz <emmunozp@microsoft.com>

* fix luis-cross train does not find imported files with locale (#1209)

* fix import file with locale

* simplify test

* change folder structures

* remove only

* add cross train config

* fix spelling

* modify test

* change naming

* fix readme

* fix typo

Co-authored-by: Emilio Munoz <emmunozp@microsoft.com>

Co-authored-by: Shuai Wang <shuwan@microsoft.com>
Co-authored-by: Tien Suwandy <tiens@microsoft.com>
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.

2 participants