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

chore(): monaco-editor dependency #3925

Closed
wants to merge 1 commit into from
Closed

chore(): monaco-editor dependency #3925

wants to merge 1 commit into from

Conversation

kmcfaul
Copy link
Contributor

@kmcfaul kmcfaul commented Mar 20, 2024

After bumping the docs framework version in react for the templates package, I'm running into an issue building the code-editor package. I think we do still need monaco-editor for it to build without manual dependency installation.

@kmcfaul
Copy link
Contributor Author

kmcfaul commented Mar 20, 2024

@nicolethoen Or should monaco-editor be a dependency of our react-code-editor package instead?

@patternfly-build
Copy link
Contributor

@kmcfaul kmcfaul closed this Mar 25, 2024
@kmcfaul
Copy link
Contributor Author

kmcfaul commented Mar 25, 2024

Closed because monaco-editor will be added as dependency of the code editor package.

@nicolethoen nicolethoen deleted the monaco-dep branch June 17, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants