Skip to content

Conversation

@vladimirgubarik
Copy link
Contributor

@vladimirgubarik vladimirgubarik commented May 5, 2025

Edit:

  • pybotx version
  • blinking test test__web_app__bot_command_response_accepted
  • workflow copier, ubuntu dependencies

Checklist

  • I've generated a new bot from the bot-template and checked that everything works:
    • Linters and tests have passed
    • Bot answers on /help command
    • Bot suggests available commands
  • I've written good commit message for all commits
  • I've split changes into separate commits where it's appropriate
  • I'll make a release when PR is approved

run: |
pip install copier==7.1.0
pip install --upgrade pip # Ensure pip is updated
pip install --upgrade argcomplete>=1.9.4 # Update argcomplete first

Choose a reason for hiding this comment

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

Не совсем понятно, зачем вызывать --upgrade, потому что установкаpip install argcomplete>=1.9.4 сама обновит версию, если она у тебя отсатет.

pip install pyyaml-include==1.4.1
# hardcode pydantic version to avoid dependency conflict with copier
pip install pydantic==1.10.2
pip install pyyaml-include

Choose a reason for hiding this comment

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

Аналогично, стоит фиксировать версию. Для поддержания совместимости между библиотеками.

@vladimirgubarik vladimirgubarik force-pushed the feature/bump-pybotx-version-0.73.4 branch from 14b253b to a71c9f1 Compare May 7, 2025 09:39
@vladimirgubarik vladimirgubarik force-pushed the feature/bump-pybotx-version-0.73.4 branch from a71c9f1 to bb5519e Compare May 7, 2025 10:15
@vladimirgubarik vladimirgubarik merged commit 9c58249 into master May 7, 2025
9 checks passed
@vladimirgubarik vladimirgubarik deleted the feature/bump-pybotx-version-0.73.4 branch May 7, 2025 12:12
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.

3 participants