Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/overlay tool #860

Merged
merged 46 commits into from
Jan 28, 2020
Merged

Feature/overlay tool #860

merged 46 commits into from
Jan 28, 2020

Conversation

dannyrb
Copy link
Member

@dannyrb dannyrb commented Feb 24, 2019

WIP

Playing w/ bash syntax in circleCI's environment
Make sure we're not stealing a version number that can be used for master
Tidying up comments and formatting
@codecov
Copy link

codecov bot commented Feb 24, 2019

Codecov Report

Merging #860 into master will increase coverage by 8.08%.
The diff coverage is 42.15%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #860      +/-   ##
==========================================
+ Coverage   10.77%   18.86%   +8.08%     
==========================================
  Files         210      284      +74     
  Lines        6803     8498    +1695     
  Branches     1075     1446     +371     
==========================================
+ Hits          733     1603     +870     
- Misses       5122     5715     +593     
- Partials      948     1180     +232
Impacted Files Coverage Δ
src/importInternal.js 0% <ø> (ø)
src/eventDispatchers/newImageEventDispatcher.js 20% <ø> (+20%) ⬆️
src/drawing/resetCanvasContextTransform.js 0% <ø> (ø) ⬆️
src/eventDispatchers/touchToolEventDispatcher.js 100% <ø> (ø) ⬆️
src/events.js 100% <ø> (ø) ⬆️
src/lib.js 100% <ø> (+100%) ⬆️
src/drawing/drawEllipse.js 0% <0%> (ø) ⬆️
src/drawing/drawRect.js 0% <0%> (ø) ⬆️
...ispatchers/mouseEventHandlers/mouseDownActivate.js 0% <0%> (ø) ⬆️
src/drawing/transformCanvasContext.js 0% <0%> (ø) ⬆️
... and 256 more

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 6c7581d...7ae3a8d. Read the comment docs.

@dannyrb
Copy link
Member Author

dannyrb commented Feb 24, 2019

  • Resolve scoped package conflict
  • Add proper commit messages (squash?)
  • Unit tests?

@swederik swederik merged commit 4eef070 into master Jan 28, 2020
@dannyrb
Copy link
Member Author

dannyrb commented Jan 28, 2020

🎉 This PR is included in version 4.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants