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

Conversation

@a-b-r-o-w-n
Copy link
Contributor

Description

The root cause is changes about the expected path of the runtimes directory due to moving the extensions directory up a level. This moves the directory in the electron build to match the directory structure in the repo.

Task Item

fixes #4472

@a-b-r-o-w-n
Copy link
Contributor Author

I'm going to trigger a build off of this to test, so hold of on merging until then.

@a-b-r-o-w-n
Copy link
Contributor Author

@coveralls
Copy link

coveralls commented Oct 22, 2020

Coverage Status

Coverage remained the same at 55.223% when pulling 6dcd9ec on abrown/fix-electron-runtime into 73e479b on main.

Copy link
Contributor

@tonyanziano tonyanziano left a comment

Choose a reason for hiding this comment

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

👍

@a-b-r-o-w-n
Copy link
Contributor Author

The build looks good. I'm going to merge.

@a-b-r-o-w-n a-b-r-o-w-n merged commit 647a4d5 into main Oct 22, 2020
@a-b-r-o-w-n a-b-r-o-w-n deleted the abrown/fix-electron-runtime branch October 22, 2020 19:55
@cwhitten cwhitten mentioned this pull request Nov 13, 2020
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* move location of builtin extensions in electron build

* fail copy scripts if error encountered

* update lockfiles
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.

Start bot failed due to some folder not found in nightly build Electron

4 participants