-
Notifications
You must be signed in to change notification settings - Fork 977
Fix/windows supported frontend #548
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
base: main
Are you sure you want to change the base?
Changes from all commits
dd43023
00caff9
1ada430
3dd9d8b
ad9a080
f159317
49189f2
1409e96
de88063
c9b442b
db3a9a3
714da4f
b34279f
95ad193
c9aab20
bba2669
2ef9627
f4eaa34
0fa4261
b6cfb5d
1f97906
1654b60
3d56cfa
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,36 @@ | ||||||
| name: Windows Build | ||||||
|
|
||||||
| on: | ||||||
| pull_request: | ||||||
| push: | ||||||
|
|
||||||
| jobs: | ||||||
| windows-build: | ||||||
| runs-on: windows-latest | ||||||
|
|
||||||
| steps: | ||||||
| - name: Checkout repo | ||||||
| uses: actions/checkout@v4 | ||||||
|
|
||||||
| - name: Setup Node | ||||||
| uses: actions/setup-node@v4 | ||||||
| with: | ||||||
| node-version: 18 | ||||||
|
|
||||||
| # --- Build web_core first --- | ||||||
| - name: Install web_core deps | ||||||
| working-directory: renderers/web_core | ||||||
| run: npm install | ||||||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You'll want to prefer
Suggested change
|
||||||
|
|
||||||
| - name: Build web_core | ||||||
| working-directory: renderers/web_core | ||||||
| run: npm run build | ||||||
|
|
||||||
| # --- Then build lit --- | ||||||
| - name: Install lit deps | ||||||
| working-directory: renderers/lit | ||||||
| run: npm install | ||||||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here
Suggested change
|
||||||
|
|
||||||
| - name: Build lit | ||||||
| working-directory: renderers/lit | ||||||
| run: npm run build | ||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this just be added as another job to an existing workflow file, like the web build?