[GitHub] cordova-lib issue #601: CB-13485: Test with Node8

[stevengill]

Github user stevengill commented on the issue:

https://github.com/apache/cordova-lib/pull/601
  
yeah i ran into this last time i tried to do it. 

npm 5 doesn't handle `npm link` properly. Doesn't work with our current 
link all the cordova deps strategy 


---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

Nightly build #523 for cordova has succeeded!

[Apache Jenkins Server]

Nightly build #523 for cordova has succeeded!
The latest nightly has been published and you can try it out with 'npm i -g 
cordova@nightly'

For details check build console at 
https://builds.apache.org/job/cordova-nightly/523/consoleFull

-
Jenkins for Apache Cordova

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

[GitHub] cordova-lib pull request #602: CB-12774 : Don't munge scoped plugin IDs anym...

[akdor1154]

GitHub user akdor1154 opened a pull request:

https://github.com/apache/cordova-lib/pull/602

CB-12774 : Don't munge scoped plugin IDs anymore. Comment requested.

In order to get plugins that are under an npm scope (e.g. 
`@akdor1154/some-plugin`) to install, a previous change from took the approach 
to consider such a plugin to be called (have the id) `some-plugin` instead of 
`@akdor1154/some-plugin`. This allowed scoped plugins to install and preserved 
the assumption that plugins will always be installed in 
`plugins_dir/[plugin_id]`.

However, it required special parsing logic around `npm` package IDs, and it 
broke the assumption that the `name` in an npm plugin's `package.json' would 
correspond to Cordova's idea of a plugin ID. IMO this approach is the source of 
further complexity which is not required, and is leading to weird bugs and 
special cases with scoped plugins. (see the linked issue, but there is stuff as 
basic as "`npm install` no longer works after installing a scoped plugin")

This PR changes approach - such plugins are now considered to have the id 
`@akdor1154/some-plugin`, in agreement with how npm treats such packages. This 
allows almost all special cases for scoped packages to be removed (yay). The 
key difference in behaviour as a result of this, though, is that while plugins 
are still installed in `plugins_dir/[plugin_id]`, `[plugin_id]` may no longer 
be a single directory, leading to plugin directories that look like
```
plugins
|- @akdor1154
|   |- some-plugin
|- cordova-some-other-plugin
```

Most of the logic changes in this PR are based around making this change 
work.

It's largely done how I want it, but I guess maintainers probably have 
strong opinions over whether this is the right way to go or not. Because of 
this I've left some commits in marked as TEMP that are for my own workflow. 
Please keep in mind I'll remove these. The only one you should be mindful of is 
the monkey patch to `cordova-common`; this would need to be raised in a 
separate PR to cordova-common if this change was approved in principle.


### Platforms affected
All

### What testing has been done on this change?
New end-to-end integration tests including a proper scoped plugin fixture
Unit tests on scoped plugins
Unit tests for plugman metadata

### Checklist
- [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
- [x] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB- is the JIRA ID & "android" is the platform 
affected.
- [x] Added automated test coverage as appropriate for this change.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/akdor1154/cordova-lib simple-scopes

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/602.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #602


commit 9c1c320d72bf2860f95bd9330ffd7718cdabba82
Author: Jarrad Whitaker 
Date:   2017-10-20T06:55:44Z

change scope behaviour to consider scopes to be part of the package name

commit 67dbb75fa96de8486418dd0a0bfbaa22300690d5
Author: Jarrad Whitaker 
Date:   2017-10-20T07:07:56Z

remove extraneous done callback from scope plugin tests

commit c6f681cc4df5fd4a5bd246df00a62753244c
Author: Jarrad Whitaker 
Date:   2017-10-20T08:40:08Z

add scoped plugin testcase

commit c2346e062518c639381ac167ba60a169500501eb
Author: Jarrad Whitaker 
Date:   2017-10-20T08:41:46Z

allow scoped plugins to exist in a dir structure reflecting there name, 
c.f. npm

commit 14c3f74a0d9033dfb250bc7b698e18eb226bf544
Author: Jarrad Whitaker 
Date:   2017-10-20T08:42:07Z

add an integration test to check scoped plugin add+remove

commit 0e383f9b419c5aa22bbacb456c0ff38649e1
Author: Jarrad Whitaker 
Date:   2017-10-20T08:43:14Z

TEMP lint with typescript

commit 7018c13a635b46364b34cf4378c78c60a1012f8b
Author: Jarrad Whitaker 
Date:   2017-10-20T08:42:27Z

TEMP override plugin discovery in cordova-common

commit 7aad7297f7306b380761e01017c067b31beca4a7
Author: Jarrad Whitaker 
Date:   2017-10-23T06:23:14Z

fix get_fetch_metadata to not guess plugins_dir anymore

commit 17a8cce029ede8060383f041bbc9735f841556bf
Author: Jarrad Whitaker 
Date:   2017-10-23T06:53:59Z

remove top_plugins override

commit c0b1f5caae2d75b1bec7052598bf0f5f03285827
Author: Jarrad Whitaker 

[GitHub] cordova-lib issue #601: CB-13485: Test with Node8

[akdor1154]

Github user akdor1154 commented on the issue:

https://github.com/apache/cordova-lib/pull/601
  
Ah, don't we love npm 5? I didn't expect this to break so early, I'll look 
tomorrow.


---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

[GitHub] cordova-lib pull request #601: CB-13485: Test with Node8

[akdor1154]

GitHub user akdor1154 opened a pull request:

https://github.com/apache/cordova-lib/pull/601

CB-13485: Test with Node8

### What does this PR do?
Enables testing on Node 8 with Travis, and fixes the failing tests (no 
behaviour changes seem necessary).

### What testing has been done on this change?
`npm test`

### Checklist
- [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
- [x] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB- is the JIRA ID & "android" is the platform 
affected.
- [x] Added automated test coverage as appropriate for this change.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/akdor1154/cordova-lib node8

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/601.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #601


commit 8e38e274cdd140717766fc43a4b28934f4106e64
Author: Jarrad Whitaker 
Date:   2017-10-23T23:38:55Z

Test Node 8 with Travis

commit 0a704ab17d420273d95c6ef3cc7b7774d6f3
Author: Jarrad Whitaker 
Date:   2017-10-23T23:51:26Z

fix error test on Node 8




---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

Re: [DISCUSS] cordova-ios@4.5.3 Release

[Darryl Pogue]

CB-13290 I think

On Mon, Oct 23, 2017 at 2:48 PM, Suraj Pindoria
 wrote:
>
> Good point Darryl, thanks for bringing this up! I agree that we might as
> well take care of it now rather than doing another release down the road.
> Do you know if there is already an existing issue for this? If not I will
> go ahead and file one.
>
> On Mon, Oct 23, 2017 at 2:09 PM, Darryl Pogue  wrote:
>
> > We might also want to revisit the list of Splash Screen sizes and add
> > support for the new ones:
> > https://developer.apple.com/ios/human-interface-
> > guidelines/icons-and-images/launch-screen/#static-launch-screen-images
> >
> > I don't consider that a release blocker, but if it's quick to do it might
> > have us from doing another bugfix release in short order.
> >

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

Re: [DISCUSS] cordova-ios@4.5.3 Release

[Suraj Pindoria]

Good point Darryl, thanks for bringing this up! I agree that we might as
well take care of it now rather than doing another release down the road.
Do you know if there is already an existing issue for this? If not I will
go ahead and file one.

On Mon, Oct 23, 2017 at 2:09 PM, Darryl Pogue  wrote:

> We might also want to revisit the list of Splash Screen sizes and add
> support for the new ones:
> https://developer.apple.com/ios/human-interface-
> guidelines/icons-and-images/launch-screen/#static-launch-screen-images
>
> I don't consider that a release blocker, but if it's quick to do it might
> have us from doing another bugfix release in short order.
>

Re: [DISCUSS] cordova-ios@4.5.3 Release

[Darryl Pogue]

We might also want to revisit the list of Splash Screen sizes and add
support for the new ones:
https://developer.apple.com/ios/human-interface-guidelines/icons-and-images/launch-screen/#static-launch-screen-images

I don't consider that a release blocker, but if it's quick to do it might
have us from doing another bugfix release in short order.

[DISCUSS] cordova-ios@4.5.3 Release

[Suraj Pindoria]

While rearranging the icon mapping list to make it more readable I
accidentally removed icon-40@2x, so this is a quick hotfix release to add
it back in.

If there is any reason to delay this release please let me know, otherwise
I will continue on with the release process tomorrow.

Re: Small UI Bug

[Jan Piotrowski]

Hm, am I correct to assume that your initial email included a
screenshot of a visual problem? Seems the mailing list filters these
out.

I reported a bug some time ago that could match the description in your email:
https://issues.apache.org/jira/browse/CB-13402

2017-10-23 11:53 GMT+02:00 Sridhar Yadav :
> PFA small UI bug.
>
> Gap for "Cordova App Showcase" section
>
>
> Regards,
> Sridhar
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
> For additional commands, e-mail: dev-h...@cordova.apache.org

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

Small UI Bug

[Sridhar Yadav]

PFA small UI bug.

Gap for "Cordova App Showcase" section


Regards,
Sridhar

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org