@voxgig/sdkgen-infrapack 0.0.9 → 0.0.10
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -1621,14 +1621,26 @@ const Workflow = cmp(function Workflow(props: any) {
|
|
|
1621
1621
|
File({ name: 'build.yml' }, () => {
|
|
1622
1622
|
Content(`# Generated by @voxgig/sdkgen. Do not edit.
|
|
1623
1623
|
#
|
|
1624
|
-
${provider.
|
|
1624
|
+
${'git' === provider.sdkDepKind ?
|
|
1625
1625
|
`# The ${provider.api} SDK is depended on by GIT TAG rather than taken from a\n` +
|
|
1626
1626
|
'# registry, so `npm install` resolves the tag named in package.json and\n' +
|
|
1627
|
-
'# needs git on PATH — every GitHub runner has it
|
|
1628
|
-
'
|
|
1629
|
-
|
|
1630
|
-
'#
|
|
1631
|
-
'# platform.'
|
|
1627
|
+
'# needs git on PATH — every GitHub runner has it.\n' +
|
|
1628
|
+
'#\n' +
|
|
1629
|
+
'# npm 12 refuses a git dependency by default (allow-git defaults to\n' +
|
|
1630
|
+
'# "none"), which is what --allow-git=all below opts back into. Nothing\n' +
|
|
1631
|
+
'# else is needed to build and run the offline tests on any platform.' :
|
|
1632
|
+
'release' === provider.sdkDepKind ?
|
|
1633
|
+
`# The ${provider.api} SDK is depended on by GITHUB RELEASE ASSET rather than\n` +
|
|
1634
|
+
'# taken from a registry: an npm-pack tarball attached to the tag, which\n' +
|
|
1635
|
+
'# npm installs without looking at the repository layout.\n' +
|
|
1636
|
+
'#\n' +
|
|
1637
|
+
'# npm 12 refuses a remote tarball by default (allow-remote defaults to\n' +
|
|
1638
|
+
'# "none"), which is what --allow-remote=all below opts back into. That\n' +
|
|
1639
|
+
'# flag does not travel to consumers of this package, so publishing the\n' +
|
|
1640
|
+
'# SDK remains the durable answer.' :
|
|
1641
|
+
`# The ${provider.api} SDK is a normal published dependency, so \`npm install\`\n` +
|
|
1642
|
+
'# is all that is needed to build and run the offline tests on every\n' +
|
|
1643
|
+
'# platform.'}
|
|
1632
1644
|
${!provider.liveApp ? '' : `#
|
|
1633
1645
|
# The live tests additionally need the companion server, which is only
|
|
1634
1646
|
# distributed in the SDK's source repository (it is not published). That repo
|
|
@@ -723,6 +723,11 @@ const Main = cmp(function Main(props: any) {
|
|
|
723
723
|
// CI note says so, because "npm install is all you need" stops being
|
|
724
724
|
// true the moment git or a tarball URL is in the path.
|
|
725
725
|
sdkGit: !sdkDep.startsWith('^'),
|
|
726
|
+
// WHICH non-registry kind, because the CI note has to be true: a git
|
|
727
|
+
// dependency needs git on PATH and a release asset does not, and a note
|
|
728
|
+
// that says the wrong one is worse than no note.
|
|
729
|
+
sdkDepKind: sdkDep.startsWith('github:') ? 'git' :
|
|
730
|
+
(sdkDep.startsWith('http') ? 'release' : 'npm'),
|
|
726
731
|
// THE OPT-IN A NON-REGISTRY DEPENDENCY NEEDS, or ''.
|
|
727
732
|
//
|
|
728
733
|
// npm 12 defaults `allow-git` and `allow-remote` to "none" and REFUSES
|
package/package.json
CHANGED