Skip to content

Commands overhaul #41

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Jul 31, 2020
Merged

Commands overhaul #41

merged 6 commits into from
Jul 31, 2020

Conversation

noplanman
Copy link
Member

@noplanman noplanman commented Jul 13, 2020 •
edited
Loading

  • Huge overhaul of all example commands and entrypoint files.
  • PHP 7.2+ is required now, which will be the default in core soon.
  • Configuration has been moved to a dedicated config.php to simplify the setup. The config array is the same as the one used for the manager, to keep things simple.
  • PHPCS has been added to ensure a clean coding style.

@noplanman noplanman requested review from Hitmare, jacklul and chuv1 July 13, 2020 12:17
@noplanman noplanman force-pushed the commands_overhaul branch from 52bb5c7 to abf1c4f Compare July 31, 2020 15:52
@noplanman noplanman merged commit d29a7b2 into php-telegram-bot:develop Jul 31, 2020
@noplanman noplanman deleted the commands_overhaul branch July 31, 2020 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hitmare Hitmare Awaiting requested review from Hitmare

@jacklul jacklul Awaiting requested review from jacklul

@chuv1 chuv1 Awaiting requested review from chuv1

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@noplanman