apm icon indicating copy to clipboard operation
apm copied to clipboard

Problem converting what appears to be a valid tmTheme file

Open hulbert opened this issue 8 years ago • 0 comments

Prerequisites

  • [x] Put an X between the brackets on this line if you have done all of the following:
    • Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
    • Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
    • Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
    • Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
    • Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description

When running the app command to convert a .tmTheme file I get an error that seems incorrect based on the error message (i.e. affect these settings are all in the array though I admit limited knowledge of the .tmTheme syntax).

This is just an adapted older version of TextMate's Solarized Light theme and works in TextMate 2.

$ apm init --theme scott-solarized-light --convert ~/Downloads/scott-solarized-light.tmTheme 
Could not find the required color settings in the theme.

The theme being converted must contain a settings array with all of the following keys:
  * background
  * caret
  * foreground
  * invisibles
  * lineHighlight
  * selection

Steps to Reproduce

  1. Run above command with this file (renamed for Github) scott-solarized-light.tmTheme.txt

Versions

$ apm -v
apm  1.18.2
npm  3.10.10
node 6.9.5 x64
python 2.7.10
git 2.10.1

hulbert avatar Aug 09 '17 10:08 hulbert