Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Composer install (PARTIAL) #2353

Merged
merged 46 commits into from
Mar 10, 2019
Merged

Conversation

mapkyca
Copy link
Member

@mapkyca mapkyca commented Feb 24, 2019

Here's what I fixed or added:

Moving what I can from /external over to composer

Here's why I did it:

Because it's $CurrentYear, and there are advantages.

This is a PARTIAL conversion, converting most stuff to composer. Remaining stuff is either customised by us, has no clear version to migrate to, or (like mongo) needs some shims on our side to work correctly. So, a migration plan needs to be worked out for these.

Refs #1064

Checklist:

  • This pull request addresses a single issue
  • If this code includes interface changes, I've included screenshots in this Pull Request thread
  • I've adhered to Known's style guide
  • My git branch is named in a descriptive way - i.e., yourname-summary-of-issue
  • I've tested my code in-browser
  • My code contains descriptive comments
  • I've added tests where applicable

This is the start of moving dependency modules over to composer.
Removing manual load of php-ogp, leaving this for PSR-4 autoloader
Adding PSR Logging library
Adding MF2 parser, via composer
Addung Webignition requirements
Ensuring travis installs dependencies.
Rolling Webignition back to 2.0, since 3.0 is "in no way backwards compatible", and needs PHP 7.2
Using JQuery 3.2.1 from Components repository
Second thoughts, lets require specific versions so that upgrades can be managed
Adding HTMLPurifier library
Installing h5f via composer (from git)

This might be a deletion candidate, see idno#2354
Adding JQuery-timeago.js via composer
Adding TinyMCE, pegged to specific version.
Loading symfony class loader from composer
Loading symfony event dispatcher from composer
Loading symfony console from composer
Loading symfony components from composer
Loading SimplePie via composer
@mapkyca mapkyca changed the title Composer install (WIP) Composer install (PARTIAL) Feb 24, 2019
Replacing customised AutoP with adopted one.
Mentioning composer when installing directly off of github
Installing version 1.6.0 of underscore.js from github repository
Installing version 1.0.3 of Paypal's accessibility plugin.
Removing IE HTML shiv, since it hasn't been present for a long time.
Installing specific bootstrap toggle version from github
Installing exif-js from GitHub, rolling up to v2.3.0
Installing mediaelement 4.2.6 from github
@mapkyca
Copy link
Member Author

mapkyca commented Feb 25, 2019

All changes that are easy to do have been done - other projects don't have clear versions to use.

Adding support for bower / npm packages, installed via composer.
Tidying up the composer repo by removing the custom repository
Tidying up the composer repo by removing the custom repository
Tidying up the composer repo by removing the custom repository
Tidying up the composer repo by removing the custom repository
Tidying up the composer repo by removing the custom repository
Tidying up the composer repo by removing the custom repository
Tidying up the composer repo by removing the custom repository
Tidying up the composer repo by removing the custom repository
Tidying up the composer repo by removing the custom repository
Copy link
Member

@benwerd benwerd left a comment

Choose a reason for hiding this comment

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

I think we need to have a revamped README / installation documentation to go along with this?

@mapkyca
Copy link
Member Author

mapkyca commented Mar 10, 2019

Hmm... I updated the installation docs. Most people seem to install from packages, and for tech users I've got a fatal error that prompts for composer install.

Probably good to have in the readme as well...

@mapkyca
Copy link
Member Author

mapkyca commented Mar 10, 2019

Updated

@benwerd
Copy link
Member

benwerd commented Mar 10, 2019

I think we’ll need to say more - this is literally the bare minimum we could say - but this will work for now.

@mapkyca
Copy link
Member Author

mapkyca commented Mar 10, 2019

Weirdly git seems to have screwed up, and I made the changes but cant push them.

@mapkyca mapkyca merged commit 3e810c7 into idno:master Mar 10, 2019
@mapkyca
Copy link
Member Author

mapkyca commented Mar 10, 2019

Come back to the docs - though non-tech users are going to use premade packages

@mapkyca mapkyca deleted the mapkyca-composer-begin branch May 21, 2019 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants