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

Comments

Remove $id from schema:merge and enable authoringKey in luis:build#744

Merged
chrimc62 merged 2 commits intomasterfrom
chrimc/id
May 4, 2020
Merged

Remove $id from schema:merge and enable authoringKey in luis:build#744
chrimc62 merged 2 commits intomasterfrom
chrimc/id

Conversation

@chrimc62
Copy link

@chrimc62 chrimc62 commented May 4, 2020

Enable authoringKey config to work for luis:build. Fixes #743
Remove $id from schema:merge so that dotnet test does not modify files. Fixes #742

Enable authoringKey config to work for luis:build.
@chrimc62 chrimc62 requested a review from munozemilio May 4, 2020 20:47
@chrimc62 chrimc62 requested a review from cleemullins as a code owner May 4, 2020 20:47
"module": "commonjs",
"outDir": "lib",
"rootDir": "src",
"sourceMap": true,
Copy link
Contributor

Choose a reason for hiding this comment

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

is this needed?

@cleemullins
Copy link
Contributor

Build is failing. Any idea why?

@chrimc62 chrimc62 merged commit d2e756a into master May 4, 2020
@chrimc62 chrimc62 deleted the chrimc/id branch May 4, 2020 21:15
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.

luis:build does not pick up config authoringKey dialog:merge should not include $id

3 participants