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

fix(build): build with ng-packagr #4617

Merged
merged 4 commits into from
Oct 15, 2018
Merged

Conversation

Domainv
Copy link
Contributor

@Domainv Domainv commented Sep 17, 2018

Closes #4535 #4495 #4494 #4413 #4380 #4552

screen shot 2018-09-27 at 9 10 32 pm
screen shot 2018-09-27 at 9 11 15 pm

After adding new ngx-bootstrap version (build with ng-packagr)
screen shot 2018-09-27 at 9 11 34 pm

PR Checklist

Before creating new PR, please take a look at checklist below to make sure that you've done everything that needs to be done before we can merge it.

  • read and followed the CONTRIBUTING.md guide.
  • built and tested the changes locally.
  • added/updated tests.
  • added/updated API documentation.
  • added/updated demos.

@ghost ghost assigned Domainv Sep 17, 2018
@ghost ghost added the needs review label Sep 17, 2018
@Domainv Domainv force-pushed the feat-build-with-packagr branch 8 times, most recently from a075d5d to a544620 Compare September 21, 2018 18:52
@Domainv Domainv force-pushed the feat-build-with-packagr branch from a544620 to e767b64 Compare September 26, 2018 20:25
@codecov
Copy link

codecov bot commented Sep 26, 2018

Codecov Report

Merging #4617 into development will increase coverage by 2.87%.
The diff coverage is 100%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #4617      +/-   ##
===============================================
+ Coverage        71.48%   74.35%   +2.87%     
===============================================
  Files              259      202      -57     
  Lines             8384     7101    -1283     
  Branches          1562     1446     -116     
===============================================
- Hits              5993     5280     -713     
+ Misses            1985     1426     -559     
+ Partials           406      395      -11
Impacted Files Coverage Δ
...themes/bs/bs-datepicker-day-decorator.directive.ts 100% <ø> (ø) ⬆️
src/component-loader/component-loader.factory.ts 100% <ø> (ø) ⬆️
src/component-loader/component-loader.class.ts 71.22% <ø> (-2.98%) ⬇️
src/utils/triggers.ts 60.71% <ø> (-2.62%) ⬇️
...emes/bs/bs-datepicker-navigation-view.component.ts 91.66% <ø> (ø) ⬆️
src/timepicker/reducer/timepicker.actions.ts 100% <ø> (ø) ⬆️
src/dropdown/bs-dropdown.state.ts 60% <ø> (-40%) ⬇️
src/timepicker/reducer/timepicker.reducer.ts 76.74% <ø> (ø) ⬆️
src/datepicker/models/index.ts 100% <ø> (ø) ⬆️
src/pagination/pager.component.ts 76.98% <ø> (-3.18%) ⬇️
... and 90 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 63af3e1...7d1b88d. Read the comment docs.

@Domainv Domainv force-pushed the feat-build-with-packagr branch 2 times, most recently from 4904da6 to 751aa49 Compare September 27, 2018 16:05
@Domainv Domainv changed the title WIP fix(build): build with ng-packagr fix(build): build with ng-packagr Sep 27, 2018
@Domainv Domainv force-pushed the feat-build-with-packagr branch 3 times, most recently from 6ce20eb to e5ab73d Compare September 27, 2018 18:57
@ghost ghost assigned valorkin Sep 27, 2018
@OlegDubrovskyi
Copy link
Contributor

Looks good.
Smoke testing, Chrome 78.

@Domainv Domainv force-pushed the feat-build-with-packagr branch 3 times, most recently from a49808a to 1b59ca0 Compare October 11, 2018 14:58
@Domainv Domainv force-pushed the feat-build-with-packagr branch from 2cf5ebd to fa2817f Compare October 12, 2018 13:33
@ghost ghost added the needs review label Oct 12, 2018
@OlegDubrovskyi
Copy link
Contributor

Looks good.
Smoke test.
Chrome.

@valorkin valorkin merged commit 90765c9 into development Oct 15, 2018
@ghost ghost removed the ready for merge label Oct 15, 2018
@Domainv Domainv deleted the feat-build-with-packagr branch October 17, 2018 09:04
leo6104 pushed a commit to leo6104/ngx-bootstrap that referenced this pull request Oct 10, 2019
* fix(build): build with ng-packagr

* chore(changelog): update [skip ci]

* fix(changelog): changelog revert
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