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

KOGITO-5725: Enable extensions for github.dev #617

Merged
merged 1 commit into from
Sep 17, 2021
Merged

KOGITO-5725: Enable extensions for github.dev #617

merged 1 commit into from
Sep 17, 2021

Conversation

caponetto
Copy link
Contributor

This PR includes all necessary changes to run the extensions on github.dev. Details are described in the official guide.

Extensions that will be enabled:

  • vscode-extension-bpmn-editor (prod)
  • vscode-extension-dmn-editor (prod)
  • vscode-extension-pmml-editor (prod)
  • vscode-extension-pack-kogito-kie-editors (dev)

Note: Bundles cannot be enabled.

The way to test the extensions on the browser is to use the @vscode/test-web package.
It basically opens VS Code on the browser, allowing tests closer to the actual github.dev env.
To run this, use the new yarn run:webmode script that is available for each extension.

Known issues: The Resource Content API does not seem to be working on this vscode-test-web env.
This means that importing WIDs (BPMN), included models (DMN), and search for DMNs (SCESIM) can't be validated.
There's an open issue to better understand this behavior.
Also, I wasn't able to resize the window and I've seen some issues with importing folders.

Anyway, the most important thing here is that, after these changes, the extensions keep working as expected on the desktop.

@caponetto caponetto changed the title KOGITO-5725: Enable extensions for github.dev [WIP] KOGITO-5725: Enable extensions for github.dev Sep 6, 2021
@caponetto caponetto changed the title [WIP] KOGITO-5725: Enable extensions for github.dev KOGITO-5725: Enable extensions for github.dev Sep 6, 2021
@caponetto caponetto added the pr: wip PR is still under development label Sep 6, 2021
@caponetto caponetto removed the pr: wip PR is still under development label Sep 6, 2021
@tomasdavidorg
Copy link
Contributor

Hi @caponetto , I am sorry but I don't have time to take a look on this PR this week. I also asked others from QE and they are also busy. I can take a look next week.

@jomarko
Copy link
Contributor

jomarko commented Sep 17, 2021

Manual sanity checks

  • open two DMN files
  • open two BPMN files in parallel
  • open two SCESIM files in parallel
  • open two PMML file sin parallel
  • DMN include DMN
  • DMN include PMML
  • SCESIM - test DMN
  • ??? generate SVG ???

Copy link
Contributor

@jomarko jomarko left a comment

Choose a reason for hiding this comment

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

Sanity checks of the vscode kogito extension passed. no deep checks of webmode done in the time of providing this approval.

@tiagobento tiagobento merged commit dc393fe into apache:main Sep 17, 2021
vpellegrino pushed a commit to vpellegrino/kie-tools that referenced this pull request Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants