Skip to content
This repository has been archived by the owner on Feb 27, 2020. It is now read-only.

@TODO #1

Open
Alexia opened this issue May 22, 2015 · 6 comments
Open

@TODO #1

Alexia opened this issue May 22, 2015 · 6 comments

Comments

@Alexia
Copy link
Owner

Alexia commented May 22, 2015

  • Localization
    • Setup localization functionality for all messages; especially obscure one.
    • Use JSON
    • Add option switch
    • Suggestion: Use the l(/.../) format.
  • CLI Text Output Handler
  • Editor Support
    • Add linking support for editors that support it such as "tmxt:https://" and "subl:https://".
  • Composer/Packagist
  • Option for different file extensions Thanks, @willbonde
@kherge
Copy link

kherge commented May 22, 2015

I'd like to recommend that support for Composer be added, and that the package be made available through Packagist.

@Alexia
Copy link
Owner Author

Alexia commented May 22, 2015

I really want to say no since I am not fond of Composer, but I know people will keep asking. :P I made an account earlier and will look into it.

@Alexia
Copy link
Owner Author

Alexia commented May 22, 2015

@kherge
https://packagist.org/packages/alexia/php7mar
Does that look correct to you? I setup the automatic updater hook on GitHub as well.

@Swader
Copy link

Swader commented May 25, 2015

It works, thanks:

vagrant@homestead:~/Code/diffbot_test$ composer global require alexia/php7mar:dev-master
Changed current directory to /home/vagrant/.composer
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
  - Installing alexia/php7mar (dev-master 38d2d9d)
    Cloning 38d2d9dd357351c21c821408c71a4bfc869a1265

Writing lock file
Generating autoload files
vagrant@homestead:~/Code/diffbot_test$ php ~/.composer/vendor/alexia/php7mar/
classes/       composer.json  .git/          .gitignore     LICENSE        mar.php        README.md      reports/       testcases.php  
vagrant@homestead:~/Code/diffbot_test$ php ~/.composer/vendor/alexia/php7mar/mar.php -f=.
Report located at: /home/vagrant/.composer/vendor/alexia/php7mar/reports/2015-05-25 06:37:33 diffbot_test.md

You might want to tag a release soon, though (even if unstable), so it can be installed without dev-master. Maybe even package it up into a phar so it can be run directly from the command line after global installation, like so:

$ php7mar -f=.

@peterbowey
Copy link

There are two others PHP 7 migration tools that I looked at, and tested - both requiring composer.

  1. https://github.com/gisostallenberg/php-to-7-aid
  2. https://github.com/Firehed/php7ize

Neither of the above were as helpful or as effective as https://github.com/Alexia/php7mar in my server environment which does not normally need to deal with "Composer", or use *.phar packages or other library boxes for a basic PHP CLI.

Thanks @Alexia for a good minimalism approach.

@Alexia
Copy link
Owner Author

Alexia commented Jul 5, 2015

Thanks. That is why the Composer package type is set to project. It allows people to install it through Composer, but it will run fine independently. :D

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants