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

Travis CI: Cache npm's $HOME/.npm folder #5279

Closed
wants to merge 512 commits into from
Closed

Conversation

ntwb
Copy link
Member

@ntwb ntwb commented Feb 27, 2018

Description

Suggested by @zkat of npm to try and cache npm's $HOME/.npm folder on Travis CI to shave of a few seconds of CI time....

How Has This Been Tested?

Travis CI performance results will be posted in follow up comments

Screenshots (jpeg or gifs if applicable):

Types of changes

CI testing performance

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code has proper inline documentation.

@ntwb ntwb added the [Type] Build Tooling Issues or PRs related to build tooling label Feb 27, 2018
@ntwb
Copy link
Member Author

ntwb commented Feb 27, 2018

This PR 1st pass:

Travis CI Job 16676.1 - Total Time: 4 min 13 sec - npm install time 54.12s
Travis CI Job 16676.4 - Total Time: 4 min 54 sec - npm install time 60.24s

This PR 2nd pass:

Travis CI Job 16676.1 - Total Time: 4 min 36 sec - npm install time 59.76s
Travis CI Job 16676.4 - Total Time: 4 min 30 sec - npm install time 52.63s

This PR 3rd pass:

Travis CI Job 16676.1 - Total Time: 4 min 26 sec - npm install time 50.13s
Travis CI Job 16676.4 - Total Time: 5 min 10 sec - npm install time 67.28s

noisysocks and others added 28 commits March 12, 2018 13:11
Move bindTitleRef so that the React lifecycle methods immediately follow
the constructor.
There's no need, as we're passed props as the constructor's first
argument.
Simplify the edit panel DOM tree by replacing <span><b /></span> with
just the single <b>.
This fixes #5516, but is blocked from merge by #5496, as it uses a newer sprite than is merged into Dashicons.
Default to content-box box-sizing for the metabox area.
* Writing Flow: Cancel isTyping when focusing non text field

* Writing Flow: Wait to determine typing clear on focus
* Keyboard Shortcuts: Remove global escape clear selected block

This doesn't apply in most text fields, and where it does apply is often unexpected (e.g. contextual toolbar)

* Toolbar: Return focus to selection when escape pressed

* Keyboard Shortcuts: Clear multi-selection by escape
* Post Title: Convert connect usage to wordpress/data

* Post Title: Insert initial block as provisional
* i18n: Absorb errors from Jed localization

* i18n: Allow setting locale data by domain
* Revert "Show the block appender even if the last block is non empty paragraph"

This reverts commit 5bb5310.

* Writing Flow: Redirect click below editor to last text field
…5553)

* Components: Enhance KeyboardShortcuts to accept children

* State: Override state on edits to same post property

* Title: Override history of textarea to dispatch undo
Alexander Botteram and others added 26 commits March 28, 2018 11:13
…ia-label-ie11-jaws

Add an aria-label to the inserter items to make IE11 and JAWS happy.
Summary of button labels and flows:

Editors and admins:

Update — updates immediately, no pos-publish flow
Publish... — opens post-publish flow
Schedule... — opens post-publish flow
Contributors:

Publish... — opens post-publish flow where the confirm button says "Submit for Review".
* Blocks: Extend deprecation timeline for Editable to 2.6

Documentation page is still active, despite having been removed: https://wordpress.org/gutenberg/handbook/block-api/editable-api/

* Framework: Remove deprecations slated for 2.5 removal
Updated documentation for PluginMoreMenuItem component
* Plugins: Subscribe to plugin registration via action

Co-authored-by: Alexander Botteram <alexander@yoast.com>

* Plugins: Fix hook name for unregistration
…n-text

Add visible text to the Gallery add item button
The test case "should return true for posts with status future" was invalid as besides the status being future the scheduled date needs also to be in the future otherwise the post is not scheduled but published.
And fix `svn st -> svn add` pipe with `xargs`
Docs: Release: Fix URL for recently closed PRs; update changelog usage
@ntwb
Copy link
Member Author

ntwb commented Mar 29, 2018

lol, that rebase escalated quickly...

@ntwb ntwb closed this Mar 29, 2018
@ntwb ntwb deleted the try/travis-npm-cache branch March 29, 2018 00:54
@ntwb ntwb mentioned this pull request Mar 29, 2018
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Build Tooling Issues or PRs related to build tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.