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

Add example project #73

Merged
merged 3 commits into from
Mar 30, 2018
Merged

Conversation

karroffel
Copy link
Member

@karroffel karroffel commented Mar 30, 2018

This adds a new example project, hello_world, which contains the Rust sources as well as a Godot project that's ready to be used (apart from possibly adjusting the GDNativeLibrary file).

Further, it moves the previous example into examples/misc_mesh_playground, where also a valid Godot project was added.

@karroffel karroffel force-pushed the add-example-project branch from 62b12d2 to 6a9e188 Compare March 30, 2018 12:30
@nical nical merged commit b707692 into godot-rust:master Mar 30, 2018
bors bot added a commit that referenced this pull request Oct 14, 2021
793: Fix clippy lints in nightly r=Bromeon a=Bromeon

Where "fix" means mostly "disable".

Lints turned off globally for the time being (might change in the future):
* `clippy::missing_safety_doc` -- requires each unsafe fn/trait to have a `# Safety` section.
   Nice in theory, annoying for internals, and boilerplate-enforcing for obvious public cases.
* `clippy::if_then_panic` -- suggests that `if (!cond) { panic!(msg); }` be written as `assert!(cond, msg);`
   Probably makes sense to enable later, but currently not possible due to [approx/#73](brendanzab/approx#73).

Lints turned of locally:
* `clippy::redundant_closure` -- works around false positive [rust-clippy/#7812](rust-lang/rust-clippy#7812)
* `dead_code` (compiler warning, not clippy) -- generated field `this` not always used. Naming it `_this` doesn't seem right, either.

Co-authored-by: Jan Haller <[email protected]>
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.

2 participants