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

Collection of various refactoring and fixes. #7

Closed
wants to merge 12 commits into from

Conversation

pointybeard
Copy link

@pointybeard pointybeard commented Oct 8, 2018

This includes a number of changes from myself as well as a PR#1 from vbartusevicius:

  • fix: Update Lexer to extend AbstractLexer
  • chore: Add .phpunit.result.cache to .gitignore
  • chore: Update composer library versions and use psr-4 autoloader
  • chore: Add PHP 7.4 and 8.0 to .travis.yml
  • fix: Incorrect namespace for Doctrine Lexer base class
  • refactor: Update tests to use assertMatchesRegularExpression() instead of deprecated assertRegExp()
  • refactor: Remove use of abandoned patchwork/utf8 library.
  • fix: Propagate the call to mt_rand() by assigning it to $seed (Fixes SimpleRandom constructor does not set seed value randomly if $seed is null #6)
  • use PHP_INT_MAX for largest possible max instead of hardcoded value

@pointybeard pointybeard changed the title Propagate the call to mt_rand() by assigning it to seed argument (Fix… Collection of various refactoring and fixes. Nov 14, 2021
@pointybeard pointybeard deleted the branch icomefromthenet:master November 14, 2021 08:18
@pointybeard pointybeard deleted the master branch November 14, 2021 08:18
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.

SimpleRandom constructor does not set seed value randomly if $seed is null
1 participant