Version 1.6.0, PTB 0.70.*, remove logging, various code tweaks.

Author: noplanman

.scrutinizer.yml

@@ -1,4 +1,6 @@
 build:
+  environment:
+    php: 7.3.0
   nodes:
     analysis:
       tests:
@@ -8,28 +10,3 @@ build:
 filter:
   paths:
     - src/
-
-checks:
-  php:
-    check_method_contracts:
-      verify_interface_like_constraints: true
-      verify_documented_constraints: true
-      verify_parent_constraints: true
-    fix_doc_comments: true
-    fix_identation_4spaces: true
-    fix_line_ending: true
-    fix_linefeed: true
-    fix_php_opening_tag: true
-    fix_use_statements:
-      remove_unused: true
-      preserve_multiple: false
-      preserve_blanklines: true
-      order_alphabetically: true
-    more_specific_types_in_doc_comments: true
-    no_goto: true
-    param_doc_comment_if_not_inferrable: true
-    parameter_doc_comments: true
-    remove_extra_empty_lines: true
-    return_doc_comment_if_not_inferrable: true
-    return_doc_comments: true
-    simplify_boolean_return: true

.travis.yml

@@ -1,27 +1,23 @@
-sudo: required
+dist: bionic
 language: php
 
 addons:
-  mariadb: 10.1
+  mariadb: 10.2
 
 cache:
   directories:
     - vendor
     - $HOME/.composer/cache
 
-matrix:
-  include:
-    - php: 7.3
-    - php: 7.2
-    - php: 7.1
-      env: COVERAGE=1
-    - php: 7.1
-      env: DEPS=low
-    - php: nightly
-    - php: hhvm
+php:
+  - 7.3
+  - 7.4
+  - 8.0snapshot
+  - nightly
+
+jobs:
   allow_failures:
     - php: nightly
-    - php: hhvm
   fast_finish: true
 
 notifications:
@@ -36,12 +32,11 @@ git:
   depth: 1
 
 install:
-  - if [ "$DEPS" == "low" ]; then travis_retry composer update --prefer-source --prefer-lowest --prefer-stable --no-interaction; fi
-  - if [ "$DEPS" != "low" ]; then travis_retry composer install --prefer-source --no-interaction; fi
+  - travis_retry composer install --prefer-dist --no-interaction
 
 before_script:
   - mysql -u root -e 'create database telegrambot; use telegrambot; source ./vendor/longman/telegram-bot/structure.sql;'
-  - if [[ "$COVERAGE" == "1" ]] && [[ "$TRAVIS_PULL_REQUEST" == "false" ]]; then CODECOV="1"; fi
+  - if [[ "$TRAVIS_PHP_VERSION" == "7.3" ]] && [[ "$TRAVIS_PULL_REQUEST" == "false" ]]; then CODECOV="1"; fi
 
 script:
   - composer check-code

CHANGELOG.md

@@ -14,6 +14,18 @@ Exclamation symbols (:exclamation:) note something of importance e.g. breaking c
 ### Fixed
 ### Security
 
+## [1.6.0] - 2020-12-26
+### Notes
+- [:ledger: View file changes][1.6.0]
+### Changed
+- Upgraded dependencies and bumped core to version 0.70.*.
+- Upgrade code to PHP 7.3.
+- Update Travis-CI and Scrutinizer configs.
+### Removed
+- [:exclamation:][1.6.0-bc-remove-botmanager-initlogging] Remove `BotManager::initLogging()` method and require separate logging initialisation with `TelegramLog::initialize`.
+### Security
+- Minimum PHP 7.3, allow PHP 8.0.
+
 ## [1.5.0] - 2019-07-29
 ### Notes
 - [:ledger: View file changes][1.5.0]
@@ -54,7 +66,7 @@ Exclamation symbols (:exclamation:) note something of importance e.g. breaking c
 ### Notes
 - [:ledger: View file changes][1.2.2]
 ### Added
-- Linked version numbers in changelog for easy verification of code changes. 
+- Linked version numbers in changelog for easy verification of code changes.
 ### Changed
 - Upgraded dependencies and bumped core to version 0.48.0.
 
@@ -187,10 +199,12 @@ Exclamation symbols (:exclamation:) note something of importance e.g. breaking c
 
 [Tidelift]: https://tidelift.com/subscription/pkg/packagist-php-telegram-bot-telegram-bot-manager?utm_source=packagist-php-telegram-bot-telegram-bot-manager&utm_medium=referral&utm_campaign=changelog
 
+[1.6.0-bc-remove-botmanager-initlogging]: https://github.com/php-telegram-bot/telegram-bot-manager/wiki/Breaking-backwards-compatibility#remove-botmanagerinitlogging "Remove BotManager::initLogging()"
 [1.0.0-bc-move]: https://github.com/php-telegram-bot/telegram-bot-manager/wiki/Breaking-backwards-compatibility#namespace-and-package-name-changed "Namespace and package name changed"
 [0.44.0-bc-parameter-structure]: https://github.com/php-telegram-bot/telegram-bot-manager/wiki/Breaking-backwards-compatibility#parameter-structure-changed "Parameter structure changed"
 
 [Unreleased]: https://github.com/php-telegram-bot/telegram-bot-manager/compare/master...develop
+[1.6.0]: https://github.com/php-telegram-bot/telegram-bot-manager/compare/1.5.0...1.6.0
 [1.5.0]: https://github.com/php-telegram-bot/telegram-bot-manager/compare/1.4.0...1.5.0
 [1.4.0]: https://github.com/php-telegram-bot/telegram-bot-manager/compare/1.3.0...1.4.0
 [1.3.0]: https://github.com/php-telegram-bot/telegram-bot-manager/compare/1.2.2...1.3.0

README.md

@@ -24,7 +24,7 @@ Installation and usage is pretty straight forward:
 Either run this command in your command line:
 
 ```bash
-composer require php-telegram-bot/telegram-bot-manager:^1.4
+composer require php-telegram-bot/telegram-bot-manager:^1.6
 ```
 
 **or**
@@ -33,7 +33,7 @@ For existing Composer projects, edit your project's `composer.json` file to requ
 
 ```yaml
 "require": {
-    "php-telegram-bot/telegram-bot-manager": "^1.4"
+    "php-telegram-bot/telegram-bot-manager": "^1.6"
 }
 ```
 and then run `composer update`
@@ -46,12 +46,12 @@ It is possible however, to override the core version that this library requires:
 
 ```yaml
 "require": {
-    "php-telegram-bot/telegram-bot-manager": "^1.4",
-    "longman/telegram-bot": "dev-develop as 0.59"
+    "php-telegram-bot/telegram-bot-manager": "^1.6",
+    "longman/telegram-bot": "dev-develop as 0.70"
 }
 ```
 
-This example will pull the develop version of the core library, making it appear to be version 0.57, which then satisfies the requirement.
+This example will pull the develop version of the core library, making it appear to be version 0.70, which then satisfies the requirement.
 
 ### Performing actions
 
@@ -176,7 +176,7 @@ $bot = new BotManager([
 
 Apart from the necessary API key, the bot can be easily configured using extra parameters.
 
-Set the webhook? Enable admins? Add custom command paths? Set up logging?
+Set the webhook? Enable admins? Add custom command paths?
 
 **All no problem!**
 
@@ -224,16 +224,6 @@ $bot = new BotManager([
         '1.1.1.1-2.2.2.2', // range
     ],
 
-    // (array) Paths where the log files should be put.
-    'logging'          => [
-        // (string) Log file for all incoming update requests.
-        'update' => __DIR__ . '/php-telegram-bot-update.log',
-        // (string) Log file for debug purposes.
-        'debug'  => __DIR__ . '/php-telegram-bot-debug.log',
-        // (string) Log file for all errors.
-        'error'  => __DIR__ . '/php-telegram-bot-error.log',
-    ],
-
     // (array) All options that have to do with the limiter.
     'limiter'          => [
         // (bool) Enable or disable the limiter functionality.
@@ -292,7 +282,6 @@ $bot = new BotManager([
             'maintenance' => [
                 '/db_cleanup',
                 '/db_repair',
-                '/log_rotate',
                 '/message_admins Maintenance completed',
             ],
         ],

composer.json

@@ -18,15 +18,15 @@
         }
     ],
     "require": {
-        "php": "^7.1",
-        "longman/telegram-bot": "^0.59",
+        "php": "^7.3|^8.0",
+        "longman/telegram-bot": "^0.70",
         "longman/ip-tools": "^1.2",
         "psr/log": "^1.1"
     },
     "require-dev": {
-        "jakub-onderka/php-parallel-lint": "^1.0",
-        "phpunit/phpunit": "^7.5|^8.2",
-        "squizlabs/php_codesniffer": "^3.4"
+        "php-parallel-lint/php-parallel-lint": "^1.2",
+        "phpunit/phpunit": "^9.3",
+        "squizlabs/php_codesniffer": "^3.5"
     },
     "autoload": {
         "psr-4": {
@@ -41,7 +41,7 @@
     "scripts": {
         "check-code": [
             "vendor/bin/parallel-lint . --exclude vendor",
-            "vendor/bin/phpcs -snp src/ tests/"
+            "vendor/bin/phpcs"
         ],
         "test": [
             "vendor/bin/phpunit --exclude-group live"