Skip to content

Conversation

@TimothyBJacobs
Copy link
Member

Description

Removes the files block in package.json.

This hides the src/ files when installing the package via NPM which makes it more annoying to figure out how the package works. This block also isn't included for any other packages.

See https://wordpress.slack.com/archives/C02QB2JS7/p1543347395368100

How has this been tested?

I'm not really sure how to test this.

Types of changes

Bug fix

Checklist:

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

Copy link
Member

@gziolo gziolo left a comment

Choose a reason for hiding this comment

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

Thanks for submitting patch 👍

@gziolo gziolo added the [Package] Notices /packages/notices label Nov 30, 2018
@gziolo gziolo added this to the 4.7 milestone Nov 30, 2018
@gziolo gziolo merged commit e080bee into WordPress:master Nov 30, 2018
daniloercoli added a commit that referenced this pull request Nov 30, 2018
…rnmobile/danilo-try-to-fix-undo-redo

* 'master' of https://github.com/WordPress/gutenberg:
  Autocompleters: Consider block category (#12287)
  Only init TinyMCE once per instance (#12386)
  RichText: convert HTML formatting whitespace to spaces (#12166)
  Notices: Remove "files" block in package.json (#12438)
  Edit Post: Avoid rendering AdminNotices compatibility component (#12444)
  Correct the docs manifest (#12411)
daniloercoli added a commit that referenced this pull request Nov 30, 2018
…HEAD

* 'master' of https://github.com/WordPress/gutenberg:
  [RNmobile] Fix problems with undo/redo on Android (#12417)
  Add registry param to withDispatch component (#11851)
  Autocompleters: Consider block category (#12287)
  Only init TinyMCE once per instance (#12386)
  RichText: convert HTML formatting whitespace to spaces (#12166)
  Notices: Remove "files" block in package.json (#12438)
  Edit Post: Avoid rendering AdminNotices compatibility component (#12444)
  Correct the docs manifest (#12411)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Package] Notices /packages/notices

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants