[Question] Do you accept pull requests for new effects?

[ccelik97]

Hi again and thanks for implementing an user-adjustable FPS limit for the AmbientLight effect. With that provided I managed to make an altered version of it that matches my preferences more closely.

Do you accept pull requests for new effect profiles here? And if so how should be the code of the new effects formatted?

[4JX]

Yeah, contributions are always welcome!

Formatting should be done with cargo fmt (your editor probably already uses that under the hood for rust projects), the rustfmt.toml config file should take care of making it have the appropriate formatting.

I don't see a fork within your repos, mind uploading one/describing what you changed?

[ccelik97]

your editor probably already uses that under the hood for rust projects

Yeah, I'm using TabNine in VS Code and I also have Prettier (didn't change any of it's settings).

Btw I had a little issue figuring out exactly what I need to build the project on Windows. Apparently GitHub Actions' Windows environment has LLVM & Clang tools all ready to go but on my machine I ended up discovering this AIO toolchain. Linux side was fine though as I had clang installed along with it's dependencies.

I don't see a fork within your repos

That's because I did my changes locally and I was waiting for you to say that contributions are welcome. :D Let me fork it then.

[ccelik97]

Done. The pull request is awaiting your approval over here: #92