Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Remove some packages from core build. #1477

Closed
probablycorey opened this issue Jan 25, 2014 · 4 comments · Fixed by #1549
Closed

Remove some packages from core build. #1477

probablycorey opened this issue Jan 25, 2014 · 4 comments · Fixed by #1549

Comments

@probablycorey
Copy link

I think we should keep the default package list simple. Mainly for three reasons.

  1. It gives people a reason to check out the package store and download packages.
  2. It keeps the menu and command palette UI uncluttered.
  3. There is a speed cost to loading extra packages.
@probablycorey
Copy link
Author

Here is a list of the load times for packages. Although they are low there is a cost to keeping all of our current packages in core.

tree-view 58ms
coffee-eval 30ms
status-bar 18ms
metrics 18ms
grammar-selector 16ms
spell-check 14ms
fuzzy-finder 12ms
snippets 11ms
metrics 11ms
wrap-guide 10ms
tabs 10ms
atom-dark-ui 10ms
terminal 8ms
feedback 8ms
status-bar 7ms
timecop 7ms
archive-view 7ms
image-view 7ms
styleguide 7ms
bracket-matcher 7ms
dev-live-reload 6ms
editor-stats 5ms
keybinding-resolver 5ms
snippets 5ms
find-and-replace 5ms
to-the-hubs 5ms
tabs 5ms
keybinding-resolver 4ms
release-notes 4ms
grammar-selector 4ms
release-notes 4ms
command-logger 4ms
git-diff 4ms
bookmarks 4ms
atom-dark-syntax 4ms
bracket-matcher 3ms
autoflow 3ms
background-tips 3ms
git-diff 3ms
settings-view 3ms
go-to-line 3ms
settings-view 3ms
github-sign-in 3ms
feedback 2ms
wrap-guide 2ms
editor-stats 2ms
link 2ms
command-palette 2ms
background-tips 2ms
visual-bell 2ms
bookmarks 2ms
symbols-view 2ms
visual-bell 2ms
gists 2ms
image-view 2ms
terminal 2ms
package-generator 2ms
styleguide 2ms
dev-live-reload 2ms
spell-check 2ms
archive-view 2ms
ascii-art 2ms
exception-reporting 1ms
welcome 1ms
tree-view 1ms
timecop 1ms
symbols-view 1ms
whitespace 1ms
markdown-preview 1ms
go-to-line 1ms
github-sign-in 1ms
fuzzy-finder 1ms
command-logger 1ms
language-git 1ms
autosave 1ms
language-sass 1ms
whitespace 1ms
language-text 1ms
autocomplete 1ms
markdown-preview 1ms
find-and-replace 1ms
autosave 1ms
ascii-art 1ms
package-generator 1ms

@kevinsawicki
Copy link
Contributor

What packages are you suggesting we remove?

@probablycorey
Copy link
Author

I'd like to remove terminal, editor-stats, visual-bell and command-logger. Timecop could probably be removed too, but I can see how it would be helpful for debugging. But we can always tell people to install the timecop package through the package store.

@lock
Copy link

lock bot commented Jan 27, 2019

This issue has been automatically locked since there has not been any recent activity after it was closed. If you can still reproduce this issue in Safe Mode then please open a new issue and fill out the entire issue template to ensure that we have enough information to address your issue. Thanks!

@lock lock bot locked as resolved and limited conversation to collaborators Jan 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants