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

Upgrade library to Angular 15 #114

Merged
merged 3 commits into from
Dec 19, 2022
Merged

Upgrade library to Angular 15 #114

merged 3 commits into from
Dec 19, 2022

Conversation

OscarDeDios
Copy link
Contributor

@OscarDeDios OscarDeDios commented Dec 13, 2022

PR Details

Update to Angular 15 and TypeScript 4.8

Description

Followed Angular Update Guide

https://update.angular.io/?l=3&v=14.0-15.0

Related Issue

#112

Motivation and Context

We want to have the latest technology.

How has this been tested

It has been executed tests and tested in a real project with npm link.

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read the CONTRIBUTING document
  • My code follows the code style of this project
  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed
  • A new branch needs to be created from master to evolve previous versions
  • Increase version in package.json following Semantic Versioning
  • Add the issue into the right project with the proper status (In progress)

@codecov
Copy link

codecov bot commented Dec 13, 2022

Codecov Report

Base: 69.16% // Head: 69.16% // No change to project coverage 👍

Coverage data is based on head (fbb3924) compared to base (ce08082).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #114   +/-   ##
=======================================
  Coverage   69.16%   69.16%           
=======================================
  Files           7        7           
  Lines         334      334           
  Branches       74       74           
=======================================
  Hits          231      231           
  Misses         86       86           
  Partials       17       17           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@OscarDeDios OscarDeDios requested review from fjaguado and removed request for fjaguado December 14, 2022 07:44
@OscarDeDios OscarDeDios merged commit b18f3aa into master Dec 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants