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

Update php version #13

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

ShadyChars
Copy link

Updating required PHP version and fixing broken tests.

OrkneyDullard and others added 18 commits April 3, 2018 22:26
* Fixed and simplified CSL Creator test
* Fixed broken Bibtex->CSL conversion test
* Moved from setExpectedException() method calls to annotations
* Added BIBFORMAT.php from OSBiB
* Rearranged OSBiB library files to more closely mirror the directory structure of the source library
* Modified BIBFORMAT to remove all non-escape-related functionality and to tighten up how it converts (probable) ISO 8859-1 strings to UTF-8 strings
* Modified BibTeX\Parser to decode LaTeX escape sequences
* Fixed some documentation
* Added missing braces around lowercase "i"s.
…reviously, they would make their way into the intermediate Entry data structures from which CSL or RIS entries would be created, but neither of those formats supports explicit casing. Thus, although we lose some information this way, the resultant strings tend to look a little nicer.
* Improved opening and closing brace replacement
* Added a "Limitations" section to README.md
phpunit (a dev dependency) needs PHP 7.0, but converter itself needs only 5.3 (as required by composer.json).

I've updated .travis.yml to match phpunit's requirements.
Not clear what phptal is used for in this context, so I've removed it entirely.
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