Conversation
c861397 to
3af04c8
Compare
Member
|
Hah, you beat me to it! I'll review this once you mark it as ready :) |
3af04c8 to
5626fdc
Compare
Collaborator
Author
haha I'm sorry (; long train ride back and got bored ^^ but I'm out of Battery now :( |
DaAlbrecht
commented
Apr 23, 2025
BD103
requested changes
Apr 25, 2025
Member
BD103
left a comment
There was a problem hiding this comment.
The concept is great, it just skips over enums :)
I made some suggestions that should cover them, but let me know if you disagree with anything or have any questions!
Collaborator
Author
omg, you are right I completely missed that ^^ |
Co-authored-by: BD103 <59022059+BD103@users.noreply.github.com>
Co-authored-by: BD103 <59022059+BD103@users.noreply.github.com>
enum variant fields
BD103
approved these changes
Apr 25, 2025
Comment on lines
+151
to
+155
| .variants | ||
| .iter() | ||
| .flat_map(|variant| variant.data.fields()) | ||
| .copied() | ||
| .collect(), |
Member
There was a problem hiding this comment.
Much cleaner than my for loop, good thinking!
BD103
added a commit
that referenced
this pull request
Apr 30, 2025
This PR prepares `bevy_lint` for v0.3.0, following the [release guide](https://github.com/TheBevyFlock/bevy_cli/blob/e85154b9bd5851a0dcac3815853ec8194d362fb3/bevy_lint/docs/how-to/release.md). Once this is marked ready for review, this will need at least one approval from a Bevy maintainer and no outstanding concerns. Once this is merged, a feature freeze will be enacted until the [post-release PR is merged](https://github.com/TheBevyFlock/bevy_cli/blob/e85154b9bd5851a0dcac3815853ec8194d362fb3/bevy_lint/docs/how-to/release.md#post-release). ## Blockers - [x] #380 - [x] #389 - [x] #391 ## For Reviewers Now is your chance to look through the [docs](https://thebevyflock.github.io/bevy_cli/bevy_lint/index.html), the changelog, and the migration guide to ensure everything is in working order. If you wish to test the linter, you may install it from this branch: ```sh rustup toolchain install nightly-2025-04-03 \ --component rustc-dev \ --component llvm-tools-preview rustup run nightly-2025-04-03 cargo install \ --git https://github.com/TheBevyFlock/bevy_cli.git \ --branch linter-v0.3.0 \ --locked \ bevy_lint ``` You're encouraged to run the linter on several projects to ensure it works correctly. Some good places to start include: - [The Engine](https://github.com/bevyengine/bevy) - [`bevy_new_2d`](https://github.com/TheBevyFlock/bevy_new_2d) - [The Official Template](https://github.com/bevyengine/bevy_github_ci_template) Thank you for all of your help! ## Next Steps - Release on Github - Announce the release on Discord, Mastodon, and other social medias - Potentially publish an announcement [blog post](https://bd103.github.io/) - TheBevyFlock/bevy_new_2d#365 --------- Co-authored-by: DAA <42379074+DaAlbrecht@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fixes #141