aagarwal1012 / Animated-Text-Kit

🔔 A flutter package to create cool and beautiful text animations. [Flutter Favorite Package]
https://animated-text-kit.web.app
MIT License
1.66k stars 304 forks source link

ptch: Add fadeInEnd and fadeOutBegin to test #213

Closed hs2361 closed 3 years ago

hs2361 commented 3 years ago

Description

Add the fadeInEnd and fadeOutBegin arguments to the FadeAnimatedTextKit widget in the test.

Issues

Fix #210.

Pull Request Process

  1. Ensure any install or build dependencies are removed before the end of the layer when doing a build.
  2. Update the README.md if needed with details of changes to the interface, this includes new environment variables, exposed ports, useful file locations and container parameters.
  3. Increase the version numbers in any examples files and the README.md to the new version that this Pull Request would represent. The versioning scheme we use is SemVer.
  4. You may merge the Pull Request in once you have the sign-off of one developer, or if you do not have permission to do that, you may request the second reviewer to merge it for you.
codecov[bot] commented 3 years ago

Codecov Report

Merging #213 (f5bcf8b) into master (1748cac) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #213   +/-   ##
=======================================
  Coverage   95.09%   95.09%           
=======================================
  Files           9        9           
  Lines         428      428           
=======================================
  Hits          407      407           
  Misses         21       21           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1748cac...f5bcf8b. Read the comment docs.

aagarwal1012 commented 3 years ago

@hs2361, please address the above comment. Try creating a new test file and add your specific test case in that file.

aagarwal1012 commented 3 years ago

We haven't got any update from @hs2361 for 5 days, so I am closing this PR.