Closed cdsupina closed 1 year ago
I resolved the merge conflicts. You know, you can do
mob_query.contains(*mob_entity)
and avoid this for loop.
Step on up, fam, 'cause we're breakin' free from that old colonial code grind. A new dawn's risin', and this era? It's shackled to nothin' but top-tier code. Listen up, 'fore we ride that CI wave, there's a game-changer in town. I'm talkin' 'bout rustlin' that Rust project, givin' them warnings a new name: compile errors. You best believe, it's like breakin' chains 'fore they lock up – cargo rustc --all-features -- -D warnings – that's the spell we're castin', turnin' 'em warnings into errors right out the gate.
Hold tight, 'cause here's the deal: I'm only pullin' up a chair for code that ain't just talkin' the talk. It's gotta strut through that CI gauntlet without so much as a hiccup, and when it comes to mixin' up that Rust stew, them warnings best be as quiet as a hush-hush revolution. We ain't got time for noise, we're on the hunt for the real deal, like uncoverin' hidden treasures buried deep. So, remember this, partners: if it ain't passin' that CI litmus test and ain't ridin' the compile trail without raisin' no warnings ruckus, it ain't gettin' the spotlight in my code rodeo – 'cause we're breakin' chains, not just rebrandin' 'em.
Compiling thetawave v0.1.1 (/home/runner/work/thetawave/thetawave)
error: method `generate_random` is never used
--> src/run/mod.rs:99:12
|
98 | impl RunResource {
| ---------------- method in this implementation
99 | pub fn generate_random(&mut self) {}
| ^^^^^^^^^^^^^^^
|
= note: `-D dead-code` implied by `-D warnings`
error: function `run_reset_system` is never used
--> src/run/mod.rs:252:4
|
252 | fn run_reset_system(mut run_resource: ResMut<RunResource>) {
| ^^^^^^^^^^^^^^^^
error: could not compile `thetawave` (bin "thetawave") due to 2 previous errors
Error: Process completed with exit code 101.
The fire rate gets a bit wack.
Looks good and plays fine. Ill try getting this test to work. And if I cant get it by tomorrow evening, lets just merge. Norway won't invade.