Перенос http-toolkit в github #1
Conversation
PI-544 Отделить код специфичный для Контура See merge request py-libs/kontur_http_toolkit_core!1
|
Есть ощущение, что стоит избавиться от директории |
There was a problem hiding this comment.
Два вопроса:
- Нужен ли пример здесь для opentelemetry?
- Если да, то нужно проверить этот пример)
.github/workflows/build-and-push.yml
Outdated
| runs-on: ubuntu-latest | ||
| environment: | ||
| name: pypi | ||
| url: https://pypi.org/p/kontur_http_toolkit_core |
There was a problem hiding this comment.
нужно подумать над названием package. Предлагаю сделать просто http_toolkit
There was a problem hiding this comment.
и что, в следующий раз менять все зависимости?
There was a problem hiding this comment.
для контуровских репозиториев у нас же останется kontur_http_toolkit с специфичным кодом для Контура и который будет зависеть от http_toolkit с github, и вроде зависимости не придётся менять)
| path: dist/ | ||
| - name: Publish distribution 📦 to PyPI | ||
| uses: pypa/gh-action-pypi-publish@release/v1 | ||
| github-release: |
There was a problem hiding this comment.
стоит ли оставлять такую job для создания релиза в github?
There was a problem hiding this comment.
Напомню, что в github можно создавать теги с помощью команды или создания релиза в web-интерфейсе. И во время второго случая эта job будет падать с ошибкой
kretatusha
requested review from
2union and
Amezhenko
and removed request for
Amezhenko
README.md
Outdated
| # kontur_http_toolkit_core |
There was a problem hiding this comment.
There was a problem hiding this comment.
перенёс сюда информацию с quickstart.md
# Conflicts: # README.md # docs/TRANSPORT.md # pyproject.toml # tox.ini
Публикация http-toolkit в open-source