Move files to src and comply with the dart package layout convention #621
Conversation
This file contains 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
CHANGELOG.md
Outdated
| @@ -11,6 +11,7 @@ | |||
| - Adding shortcut for loading Sprites and SpriteAnimation from the global cache | |||
| - Adding loading methods for the different `ParallaxComponent` parts and refactor how the delta velocity works | |||
| - Add tests for `Timer` and fix a bug where `progress` was not reported correctly | |||
| - move files to comply with the dart package layout convention | |||
There was a problem hiding this comment.
Nit: I would use structure instead of layout here
There was a problem hiding this comment.
The convention is called layout: https://dart.dev/tools/pub/package-layout
spydon
reviewed
Jan 12, 2021
| export 'src/components/mixins/single_child_particle.dart'; | ||
| export 'src/components/mixins/tapable.dart'; | ||
|
|
||
| export 'src/extensions/vector2.dart'; |
There was a problem hiding this comment.
I guess this is here and also in extensions.dart, game.dart and effects.dart since it'll be almost impossible to use anything without Vector2?
There was a problem hiding this comment.
should we make a fail extensions.dart on the extensions folder to export all our extensions? and then export that file everywhere instead of only vector2 ext
| @@ -1,6 +1,6 @@ | |||
| import 'dart:ui'; | |||
|
|
|||
| import '../../particle.dart'; | |||
| import '../../particles/particle.dart'; | |||
There was a problem hiding this comment.
Couldn't this just import lib/particles.dart or is that not according to the standard?
There was a problem hiding this comment.
It imports what needs to be imported, it doesn't need any of the particle implementations.
luanpotter
reviewed
Jan 17, 2021
renancaraujo
force-pushed
the
renan.file-organization
branch
from
afbde62 to
368f5e1
Compare
renancaraujo
force-pushed
the
renan.file-organization
branch
4 times, most recently
from
86f0f6c to
d38e88c
Compare
erickzanardo
approved these changes
Jan 19, 2021
renancaraujo
force-pushed
the
renan.file-organization
branch
from
d38e88c to
1ec2d07
Compare
luanpotter
reviewed
Jan 19, 2021
| @@ -1,6 +1,6 @@ | |||
| import 'package:test/test.dart'; | |||
|
|
|||
| import 'package:flame/memory_cache.dart'; | |||
| import 'package:flame/src/memory_cache.dart'; | |||
There was a problem hiding this comment.
does this create a warning? (I hope not!)
spydon
added a commit
that referenced
this pull request
Feb 24, 2021
* Update README to point at rc5 (#630) * Update README to point at rc5 * Update README.md * 👌 Throw exception from `parseAnchor` in `examples/widgets` instead of returning null (#632) * Updated the widgets docs (#628) Co-authored-by: Erick <erickzanardoo@gmail.com> * Removing initialDimensions and removeGesture methods (#627) * Removing initialDimensions and removeGesture methods * Removing wrongly commited folder * PR suggestion * Making SpriteComponent and SpriteAnimationComponent follow the same standard for empty constructors (#620) * Added fallback support for the web on the `SpriteBatch` API (#612) * Added fallback support for the web on the `SpriteBatch` API * Refactored the SpriteBatch class * 👌 Use `Offset` type directly in `JoystickAction.update` calculations (#631) * Move files to src and comply with the dart package layout convention (#621) * 👌 Use `Offset` type directly in `JoystickAction.update` calculations (#631) * Move files to src and comply with the dart package layout convention * Fixing widgets example Co-authored-by: Serge Matveenko <lig@countzero.co> Co-authored-by: Erick Zanardo <erickzanardoo@gmail.com> * Fix gesture detection on children (#636) * Fix gesture detection on children * Remove unused imports * positionParent should be private * Fixed comment * Moved parent to BaseComponent * Fix formatting * Initial implementation of the Composition class. (#634) * Initial implementation of the Composition class * Added decodeImageFromPixels web support * Added a `required` to GameWidget and updated the docs (#638) * Added a required to GameWidget and updated the docs * Removed entry from CHANGELOG * Release v1.0.0-rc6 (#639) * Add hitbox to PositionComponent (#618) * Move out collision detection methods * Add possibility to define a hull for PositionComponents * Add example of how to use hull with tapable * Update contains point comment * Fix contains point * Hull should be based on center position * Remove collision detection parts * Added tests * Use percentage of size instead of absolute size * Separate hull from PositionComponent * Clarify hull example * Fix formatting * Override correct method * Use mixin for hitbox * Update changelog * Rename HasHitbox to Hitbox * Clarified names * Center to edge is considered as 1.0 * Fix test * Add spaces within braces * Removed extra spaces in the braces * Add hitbox docs * Fix link * Moved point rotation to Vector2 extension * Render hitbox within extension * Fix rebase * Fix rebase * Fix formatting * Removing Util.dart and moving its remaining parts to better places (#640) * Removing Util.dart and moving its remaining parts to better places * Fixing lint * Doc fixes * Doc fixes * PR suggestions * Adapting SpriteBatchComponent constructors to other components (#643) * Adapting ParallaxComponent constructors to other components (#642) * Adapating ParallaxComponent constructors to other components * Removing wrongly commited folder * Update doc/components.md Co-authored-by: Jochum van der Ploeg <jochum@vdploeg.net> * Update doc/components.md Co-authored-by: Jochum van der Ploeg <jochum@vdploeg.net> Co-authored-by: Jochum van der Ploeg <jochum@vdploeg.net> * Updating mds with recently 0.x patches * 🏷 Null safety support Co-authored-by: Lukas Klingsbo <lukas.klingsbo@gmail.com> Co-authored-by: Jochum van der Ploeg <jochum@vdploeg.net> Co-authored-by: Erick <erickzanardoo@gmail.com> Co-authored-by: Renan <6718144+renancaraujo@users.noreply.github.com>
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.
Description
This addresses #614.
Type of change
Please delete options that are not relevant.
Checklist:
master[next]inCHANGELOG.mdflutter format