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

Use trunk for web demo #38

Merged
merged 10 commits into from
Oct 10, 2024
Merged

Use trunk for web demo #38

merged 10 commits into from
Oct 10, 2024

Conversation

Its-Just-Nans
Copy link
Contributor

@Its-Just-Nans Its-Just-Nans commented Jun 5, 2024

README.md Show resolved Hide resolved
@Its-Just-Nans
Copy link
Contributor Author

@emilk I think it's correct now ?

Btw I made this PR: https://github.com/emilk/eframe_template/pull/137/files

@Its-Just-Nans Its-Just-Nans requested a review from emilk June 5, 2024 15:59
@Its-Just-Nans
Copy link
Contributor Author

Hi @emilk

Any news on this PR and #27 ?

emilk pushed a commit to emilk/eframe_template that referenced this pull request Jun 27, 2024
##  Bug: assets have incorrect paths

Proof:

```
git clone https://github.com/emilk/eframe_template.git && lychee .
```

```
  31/31 ━━━━━━━━━━━━━━━━━━━━ Finished extracting links                                                                                              Issues found in 1 input. Find details below.

[./eframe_template/index.html]:
✗ [ERR] file:///tmp/tmp.xuDXKdo4H8/eframe_template/icon_ios_touch_192.png | Failed: Cannot find file
✗ [ERR] file:///tmp/tmp.xuDXKdo4H8/eframe_template/manifest.json | Failed: Cannot find file

🔍 31 Total (in 0s) ✅ 29 OK 🚫 2 Errors

```


## Fix

change links to these assets


## Related to

- #137
- related to
rerun-io/ewebsock#38 (comment)
.github/workflows/pages.yml Show resolved Hide resolved
example_app/.cargo/config.toml Outdated Show resolved Hide resolved
.github/workflows/pages.yml Show resolved Hide resolved
@Its-Just-Nans Its-Just-Nans requested a review from emilk July 19, 2024 21:45
Hasenfellvy added a commit to Hasenfellvy/eframe_template that referenced this pull request Aug 27, 2024
##  Bug: assets have incorrect paths

Proof:

```
git clone https://github.com/emilk/eframe_template.git && lychee .
```

```
  31/31 ━━━━━━━━━━━━━━━━━━━━ Finished extracting links                                                                                              Issues found in 1 input. Find details below.

[./eframe_template/index.html]:
✗ [ERR] file:///tmp/tmp.xuDXKdo4H8/eframe_template/icon_ios_touch_192.png | Failed: Cannot find file
✗ [ERR] file:///tmp/tmp.xuDXKdo4H8/eframe_template/manifest.json | Failed: Cannot find file

🔍 31 Total (in 0s) ✅ 29 OK 🚫 2 Errors

```


## Fix

change links to these assets


## Related to

- emilk/eframe_template#137
- related to
rerun-io/ewebsock#38 (comment)
@Its-Just-Nans
Copy link
Contributor Author

any news on it ?

@emilk

@emilk emilk added the exclude from changelog Don't mention this in CHANGELOG.md label Oct 10, 2024
@emilk emilk changed the title add trunk setup and update README Use trunk for web demo Oct 10, 2024
@emilk
Copy link
Member

emilk commented Oct 10, 2024

Thanks!

@emilk emilk merged commit 4083c86 into rerun-io:main Oct 10, 2024
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exclude from changelog Don't mention this in CHANGELOG.md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants