Skip to content

chore(ci): add --debug --verbose to stencil nightly build #29078

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 22, 2024
Merged

chore(ci): add --debug --verbose to stencil nightly build #29078

merged 1 commit into from
Feb 22, 2024

Conversation

alicewriteswrongs
Copy link
Contributor

Set the --debug and --verbose flags on the Stencil Nightly CI build.

What is the current behavior?

The Stencil nightly build doesn't provide all the information that it could!

What is the new behavior?

This sets the --debug and --verbose flags when calling npm run build in the 'Build Ionic Core with Stencil Prerelease' github action, which is used by the workflow which installs nightly Stencil builds and builds framework with it, thereby checking for regressions.

This change will just ensure that this nightly build provides a bit more information.

Does this introduce a breaking change?

  • Yes
  • No

@alicewriteswrongs alicewriteswrongs requested a review from a team as a code owner February 22, 2024 15:01
@alicewriteswrongs alicewriteswrongs requested review from mapsandapps and removed request for a team February 22, 2024 15:01
@rwaskiewicz
Copy link
Contributor

Output LGTM - https://github.com/ionic-team/ionic-framework/actions/runs/8006565642/job/21868617491

 [04:17.8]  @stencil/core
[04:18.1]  [LOCAL DEV] v4.12.3-dev.1708578095.bab3998 🌀
[04:18.1]  node 18.19.1 MEM: 126.9MB
[04:18.1]  linux, AMD EPYC 7763 64-Core Processor MEM: 126.9MB
[04:18.1]  cpus: 4, freemem: 15591MB, totalmem: 16758MB MEM: 126.9MB
[04:18.1]  compiler:
           /home/runner/work/ionic-framework/ionic-framework/core/node_modules/@stencil/core/compiler/stencil.js
           MEM: 126.9MB
[04:18.1]  build: 1708578095 MEM: 126.9MB
[04:18.3]  cache enabled, cacheDir:
           /home/runner/work/ionic-framework/ionic-framework/core/.stencil/.build
           MEM: 157.7MB
[04:18.3]  create workers, maxWorkers: 3 MEM: 158.3MB
[04:18.3]  Starting compilation in watch mode... MEM: 158.3MB
[04:21.9]  build, ionic, prod mode, started ...
[04:21.9]  start build, 2024-02-22T15:04:21 MEM: 452.0MB
[04:21.9]  cleaning 6 dirs ... MEM: 452.0MB
[04:21.9]  cleaning dirs finished in 2 ms MEM: 452.1MB
[04:21.9]  transpile started ...
[04:23.1]  load collection: ionicons,
           node_modules/ionicons/dist/collection/collection-manifest.json MEM:
           496.3MB
[04:31.7]  Transpiled modules: [
           "/home/runner/work/ionic-framework/ionic-framework/core/src/components/accordion-group/accordion-group-interface.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/global/config.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/hardware-back-button.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/browser/index.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/animation/animation-interface.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/components/menu/menu-interface.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/helpers.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/animation/animation-utils.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/animation/animation.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/components/nav/constants.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/framework-delegate.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/components/nav/view-controller.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/components/nav/nav-interface.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/transition/ios.transition.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/transition/md.transition.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/transition/index.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/animation/cubic-bezier.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/gesture/gesture-controller.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/gesture/listener.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/gesture/pointer-events.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/gesture/recognizers.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/gesture/index.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/platform.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/global/ionic-global.ts",
           "/home/runner/work/ionic-framework/ionic-framework/core/src/utils/sanitization/index.ts",
[04:31.7]  generated app types started ... MEM: 799.4MB
[04:31.7]  generated app types finished: src/components.d.ts in 23 ms MEM:
           802.0MB
[04:32.1]  transpile finished in 10.17 s
[04:32.1]  generate outputs started ... MEM: 810.3MB
[04:32.1]  getComponentAssetsCopyTasks: 1 MEM: 810.5MB
[04:32.1]  getComponentAssetsCopyTasks: 0 MEM: 810.5MB
[04:32.1]  copy started ...
[04:32.1]  generate collections started ... MEM: 810.5MB
[04:37.7]  generate custom elements started ...
[04:37.7]  generate hydrate app started ...
[04:37.7]  generate lazy started ...
[04:37.7]  generateEntryModules, 46 entryModules MEM: 1175.7MB
[04:37.7]  generate collections finished in 5.59 s MEM: 1175.8MB
[04:44.1]  copy finished (1347 files) in 12.02 s
[05:08.5]  generate custom elements finished in 30.79 s
[05:11.0]  generate hydrate app finished in 33.34 s
[05:25.4]  generate lazy finished in 47.75 s
[05:25.5]  generating react-library started ...
[05:25.5]  generate react started ... MEM: 1518.9MB
[05:25.5]  generating vue-library started ...
[05:25.5]  generate vue started ... MEM: 1518.9MB
[05:25.5]  generating angular-library started ...
[05:25.5]  generate angular proxies started ... MEM: 1518.9MB
[05:25.5]  generating angular-library started ...
[05:25.5]  generate angular proxies started ... MEM: 1518.9MB
[05:25.5]  generate angular proxies finished in 15 ms MEM: 1519.1MB
[05:25.5]  generate angular-library finished in 15 ms
[05:25.5]  generate angular proxies finished in 16 ms MEM: 1519.1MB
[05:25.5]  generate angular-library finished in 17 ms
[05:25.5]  generate vue finished in 17 ms MEM: 1519.1MB
[05:25.5]  generate vue-library finished in 17 ms
[05:25.5]  generate react finished in 18 ms MEM: 1519.1MB
[05:25.5]  generate react-library finished in 18 ms
[05:25.5]  generate types started ... MEM: 1519.1MB
[05:25.5]  generated app types started ... MEM: 1520.2MB
[05:25.5]  generated app types started ... MEM: 1524.0MB
[05:25.6]  generateAppTypes: dist/types/components.d.ts has changed MEM:
           1524.5MB
[05:25.6]  generated app types finished: dist/types/components.d.ts in 43 ms
           MEM: 1524.5MB
[05:25.6]  generateAppTypes: dist/types/components.d.ts has changed MEM:
           1524.5MB
[05:25.6]  generated app types finished: dist/types/components.d.ts in 22 ms
           MEM: 1524.5MB
[05:25.6]  generate types finished in 51 ms MEM: 1524.5MB
[05:25.6]  generate outputs finished in 53.48 s MEM: 1524.5MB
[05:25.6]  writeBuildFiles started ... MEM: 1524.5MB
[05:25.8]  in-memory-fs: data length: 16172 MEM: 1540.4MB
[05:25.8]  cache: data length: 277 MEM: 1540.4MB
[05:25.8]  writeBuildFiles finished, files wrote: 1886 in 205 ms MEM: 1540.4MB
[05:25.8]  finished build, 63864ms MEM: 1540.7MB
[05:25.8]  build finished in 63.89 s

Compared to what we have today:

[04:01.1]  @stencil/core
[04:01.4]  [LOCAL DEV] v4.12.3-dev.1708491712.d428cff 🌐
[04:01.4]  node 18.19.0
[04:01.4]  linux, AMD EPYC 7763 64-Core Processor
[04:01.4]  cpus: 4, freemem: 15575MB, totalmem: 16758MB
[04:05.2]  build, ionic, prod mode, started ...
[04:05.2]  transpile started ...
[04:15.4]  transpile finished in 10.16 s
[04:15.4]  copy started ...
[04:21.1]  generate custom elements started ...
[04:21.1]  generate hydrate app started ...
[04:21.1]  generate lazy started ...
[04:23.6]  copy finished (1347 files) in 8.22 s
[04:53.0]  generate custom elements finished in 31.99 s
[04:55.4]  generate hydrate app finished in 34.32 s
[05:06.4]  generate lazy finished in 45.31 s
[05:06.5]  generating react-library started ...
[05:06.5]  generating vue-library started ...
[05:06.5]  generating angular-library started ...
[05:06.5]  generating angular-library started ...
[05:06.5]  generate vue-library finished in 20 ms
[05:06.5]  generate angular-library finished in 20 ms
[05:06.5]  generate angular-library finished in 19 ms
[05:06.5]  generate react-library finished in 22 ms
[05:06.8]  build finished in 61.58 s

@alicewriteswrongs alicewriteswrongs force-pushed the ap/stencil-build-debug branch 2 times, most recently from 5fd51ed to 798b853 Compare February 22, 2024 15:48
mapsandapps
mapsandapps approved these changes Feb 22, 2024
View reviewed changes
Copy link
Contributor

@mapsandapps mapsandapps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good to merge once we get the builds fixed 😬

@rwaskiewicz
Copy link
Contributor

Created #29079 to fix CI, if it doesn't magically fix itself in the mean time

@rwaskiewicz
Copy link
Contributor

@alicewriteswrongs should be good to go after a rebase, #29079 landed

@alicewriteswrongs alicewriteswrongs added this pull request to the merge queue Feb 22, 2024
Merged via the queue into main with commit e440051 Feb 22, 2024
@alicewriteswrongs alicewriteswrongs deleted the ap/stencil-build-debug branch February 22, 2024 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@mapsandapps mapsandapps mapsandapps approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alicewriteswrongs @rwaskiewicz @mapsandapps