Skip to content

Conversation

msaelices
Copy link

See #507

msaelices and others added 7 commits June 12, 2019 14:11
* fix: loading of css when using webpack bundle

* chore: add custom webpack.config in order to use bundle during the build using current nativescript-dev-webpack plugin 0.21

* chore: add travis file to orchestrate the testing flow and tests

* fix: ios build in travis
@codecov-io
Copy link

codecov-io commented Jun 14, 2019

Codecov Report

Merging #508 into master will increase coverage by 0.65%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #508      +/-   ##
==========================================
+ Coverage   13.42%   14.07%   +0.65%     
==========================================
  Files          45       44       -1     
  Lines        1229     1172      -57     
  Branches      306      297       -9     
==========================================
  Hits          165      165              
+ Misses        833      784      -49     
+ Partials      231      223       -8
Impacted Files Coverage Δ
platform/nativescript/runtime/components/frame.js 0% <0%> (ø) ⬆️
platform/nativescript/runtime/components/page.js 0% <0%> (ø) ⬆️

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 8b51caa...76d46f4. Read the comment docs.

@msaelices
Copy link
Author

@VladimirAmiorkov actually this branch should not be merged into master as it's incompatible with NS5.

We think to know if this branch will be a different non-backward-compatible NS-vue version or should we add conditions and checks to make it work properly in both versions.

/CC @rigor789

@VladimirAmiorkov
Copy link
Contributor

Yes NS 6 brings many breaking changes which is why we are releasing a new major version. To me ti looks like supporting both NS 5 and NS 6 in a single code base is going to be not and easy task but its up to you.

@msaelices
Copy link
Author

Agreed. I am for maintaining two different branches, but Rigor rules

@jawa-the-hutt
Copy link

Question on this from a semver point of view as well understanding Rigor will ultimately decide. If we end up with two version of NS-Vue, one for NS 5 and one for NS 6, would it make sense to have the newer NS 6 compatible branch be a Major version change? Go to 3.x?

Other thought would be to major both and end up with NS-Vue 5.x and NS-Vue 6.x so that it was pretty clear which versions were compatible with which version of NS core. However, that is a pretty drastic change in version numbering so it may not really help with clarity.

@rigor789
Copy link
Member

@msaelices I've just talked to @sis0k0 about the release for 6.0, and the plan is to have a next or 6.0 (or similar) branch, with these changes - released as 2.4.0-alpha

For 5.x we will release 2.3.0 stable, without any of the breaking changes, and that will likely be the last release for 5.x (unless we need to release a patch)

@jawa-the-hutt it's a little tricky, because I'm trying to keep the version number somewhat semver friendly, but want to keep it similar to Vue's version. The plan is to release 3.x whenever I rewrite NSVue for Vue 3.x.

I'm not sure if releasing a breaking change in 2.4 would cause issues for people though - we'll have to be a bit careful.

@rigor789 rigor789 changed the base branch from master to next July 10, 2019 13:53
@rigor789 rigor789 merged commit be95893 into next Jul 10, 2019
@rigor789 rigor789 deleted the ns6 branch December 5, 2022 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants