Skip to content

Conversation

@MrMinimal
Copy link
Contributor

In order to make the msfs-rs project more accessible to beginners, I created a README.

@Eagle941
Copy link
Member

Eagle941 commented Aug 9, 2025

Very nice work! I would add a section on how to compile and run the examples.

@MrMinimal
Copy link
Contributor Author

@Eagle941 Thank you for looking at it, I updated the PR based on your feedback 🎉

I have no clue about building or running the other examples as they are targeting something very different from what I am trying to do.

Another PR would have to improve on their build instructions in the future.

@MrMinimal
Copy link
Contributor Author

@tracernz Thank you, is it mergable then?

README.md Outdated
background_color=0,0,255
htmlgauge00=WasmInstrument/WasmInstrument.html?wasm_module=gauge_logic.wasm&wasm_gauge=Demo, 0,0,1024,768
```
<img width="700" alt="grafik" src="https://github.com/user-attachments/assets/babe0670-51bf-4a33-8fc6-5e282667febe" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you check in this image as well?

Instead of hosting them on GitHub,
based on feedback in PR flybywiresim#37
@MrMinimal
Copy link
Contributor Author

@Gurgel100 Thanks for the review, I added the image to the repo locally. The main FBW aircraft repo had the images as a link but this also works :)

@MrMinimal
Copy link
Contributor Author

I'd love to get this merged to have it off my TODO list.

Any blockers?

@tracernz
Copy link
Member

tracernz commented Oct 3, 2025

Thanks, looks good!

@tracernz tracernz merged commit 9bc78f1 into flybywiresim:main Oct 3, 2025
1 check passed
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.

4 participants