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

[DEV-4147] Feature - Dev playground #28

Merged
merged 39 commits into from
Sep 9, 2021
Merged

Conversation

yuriyyakym
Copy link
Contributor

No description provided.

@yuriyyakym yuriyyakym added Type - Feature New feature or request Priority - High labels Aug 20, 2021
@yuriyyakym yuriyyakym self-assigned this Aug 20, 2021
@yuriyyakym yuriyyakym changed the title [] Feature - Dev playground [DEV-4147] Feature - Dev playground Aug 20, 2021
@linear
Copy link

linear bot commented Aug 20, 2021

@yuriyyakym yuriyyakym marked this pull request as ready for review September 7, 2021 13:25
packages/playground/package.json Outdated Show resolved Hide resolved
packages/playground/package.json Show resolved Hide resolved
packages/playground/package.json Outdated Show resolved Hide resolved
packages/playground/public/index.html Outdated Show resolved Hide resolved
packages/playground/src/Editor/Editor.tsx Outdated Show resolved Hide resolved
packages/slate-commons/src/lib/decorateExtensions.ts Outdated Show resolved Hide resolved
@kamilmielnik
Copy link
Contributor

Copy link
Contributor

@kamilmielnik kamilmielnik left a comment

Choose a reason for hiding this comment

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

👍 Build & Prettier are still failing though. Please address it before merging.

Comment on lines +6 to +13
-cT ANSI -t "[playground] " -l "npm start --prefix packages/playground" \
-cT ANSI -t "[slate-renderer] " -l "npm run build:watch --prefix packages/slate-renderer" \
-cT ANSI -t "[docx-cleaner] " -l "npm run build:watch --prefix packages/docx-cleaner" \
-cT ANSI -t "[slate-commons] " -l "npm run build:watch --prefix packages/slate-commons" \
-cT ANSI -t "[slate-editor] " -l "npm run build:watch --prefix packages/slate-editor" \
-cT ANSI -t "[slate-hyperscript] " -l "npm run build:watch --prefix packages/slate-hyperscript" \
-cT ANSI -t "[slate-lists] " -l "npm run build:watch --prefix packages/slate-lists" \
-cT ANSI -t "[slate-types] " -l "npm run build:watch --prefix packages/slate-types";
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should use lerna here, e.g.

lerna run build:watch --scope @prezly/slate-types

Copy link
Contributor Author

Choose a reason for hiding this comment

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

How is this better?
For this to work you would need to have lerna installed globally. Otherwise we would need to specify each watch script in root package.json.

@yuriyyakym yuriyyakym merged commit a5f4506 into master Sep 9, 2021
@yuriyyakym yuriyyakym deleted the feature/dev-playground branch September 9, 2021 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority - High Type - Feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants