You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Merge when the PR is small, useful, has no merge conflicts with master, and has passing tests
Close when the changes are non-trivial, the PR has conflicts with master, or there are failing tests in CI
If you close a PR, let the user know that we are currently moving the repository over to atom/atom so they should feel free to reopen the pull request there once the package code has been migrated.
Phase 2 - Migrate the package code
Create a fresh branch off master in your local clone of atom/atom with the name migrate-encoding-selector-package
Delete Atom's node_modules folder if you've built there before and make sure Atom's package-lock.json file is unmodified
Clone the atom/encoding-selector repo into the packages path of your local Atom repository and then delete its .git subfolder
Delete any unnecessary repo configuration files in the packages/encoding-selector folder like appveyor.yml, .travis.yml, ISSUE_TEMPLATE.md, and PULL_REQUEST_TEMPLATE.md
Remove any information from the package's README.md which will be inaccurate after the migration is complete (e.g. CI status badges or contribution instructions)
Edit Atom's package.json file to change the following:
Update the encoding-selector entry in packageDependencies to reflect the local path of the package:
Delete the encoding-selector entry in the dependencies section since it may contain a hardcoded tarball link. This will help avoid issues in the next step
Change the repository field to point to the main repo URL https://github.com/atom/atom
Run a full Atom build using script/build and verify that there are no unexpected build errors
Launch the Atom binary in the ./out folder and manually verify that the migrated package is working without any obvious issues. For example, if you're migrating a color theme, turn on that theme to make sure that it changes the editor colors as expected.
Open the Chrome Developer Tools console (Ctrl+Shift+I or Cmd+Option+I) and ensure that there are no errors written there. If there are, add them to this issue and investigate.
Update the second column of the encoding-selector row in packages/README.md to reflect the new local path for the package:
Add a label titled packages/encoding-selector to the migrated issues so that they can be found easily
Edit the original package repo's README.md (don't commit directly) to point contributors to the code's new home in atom/atom. You can use the following text:
### This package is now a part of the [core Atom repository](https://github.com/atom/atom/tree/master/packages/encoding-selector), please direct all issues and pull requests there in the future!
---
Send a PR to the encoding-selector repository with the updated README.md so that watchers will be notified of the change. You do not need to wait for reviews, it can be merged immediately.
Archive the package repository by going to its Settings page, scroll down to the "Danger Zone" section and click "Archive this Repository"
This issue tracks the steps necessary to migrate the
encoding-selectorpackage into the coreatom/atomrepository under thepackagespath:Migration Checklist
Phase 1 - Prepare repo for migration
atom/encoding-selectorrepositorySome guidelines for merging vs closing PRs:
master, and has passing testsmaster, or there are failing tests in CIIf you close a PR, let the user know that we are currently moving the repository over to
atom/atomso they should feel free to reopen the pull request there once the package code has been migrated.Phase 2 - Migrate the package code
masterin your local clone ofatom/atomwith the namemigrate-encoding-selector-packagenode_modulesfolder if you've built there before and make sure Atom'spackage-lock.jsonfile is unmodifiedatom/encoding-selectorrepo into thepackagespath of your local Atom repository and then delete its.gitsubfolderpackages/encoding-selectorfolder likeappveyor.yml,.travis.yml,ISSUE_TEMPLATE.md, andPULL_REQUEST_TEMPLATE.mdREADME.mdwhich will be inaccurate after the migration is complete (e.g. CI status badges or contribution instructions)package.jsonfile to change the following:encoding-selectorentry inpackageDependenciesto reflect the local path of the package:encoding-selectorentry in thedependenciessection since it may contain a hardcoded tarball link. This will help avoid issues in the next steprepositoryfield to point to the main repo URLhttps://github.com/atom/atomscript/buildand verify that there are no unexpected build errors./outfolder and manually verify that the migrated package is working without any obvious issues. For example, if you're migrating a color theme, turn on that theme to make sure that it changes the editor colors as expected.packages/README.mdto reflect the new local path for the package:package-lock.jsonfiles to your branch using this commit message:Phase 3 - Make sure the package works and send a PR
script/testand verify that they pass locallyatom/atomto verify that CI passesPhase 4 - Archive the
encoding-selectorrepoencoding-selectorrepo toatom/atomusing github-issue-mover or a scriptpackages/encoding-selectorto the migrated issues so that they can be found easilyatom/atom. You can use the following text:encoding-selectorrepository with the updated README.md so that watchers will be notified of the change. You do not need to wait for reviews, it can be merged immediately.Migration is now complete! 🎉