-
Notifications
You must be signed in to change notification settings - Fork 694
KIKIMR-22458 Generate changelog increment #14205
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
KIKIMR-22458 Generate changelog increment #14205
Conversation
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
import requests | ||
|
||
import os | ||
sys.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__), "../validate_pr_description"))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
может перенесем всё это в ydb/ci
директорию, что бы не делать так тут?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Окей, но сейчас на ревью еще один скрипт. Я бы тогда в отдельном PR перенесла их в ydb/ci, хорошо?
base_branch = sys.argv[3] | ||
suffix = sys.argv[4] | ||
|
||
GITHUB_TOKEN = subprocess.run(["git", "config", "--get", "github.token"], capture_output=True, text=True).stdout.strip() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
мне не очень нравится идея передавать токены через git config. как будто бы можно смотреть переменную окружения UPDATE_REPO_TOKEN
и не выставлять в экшене этот параметр.
print("PR description is valid.") | ||
return True | ||
|
||
except Exception as e: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
мне не очень нравятся такие большие try-catch c широкими исключениями, может тут лучше явно падать где-то, ведь это что-то совсем нештатное получается? или если штатное, то может в validate_pr_description_from_file обернуть этот вызов в try-catch?
⚪ Test history | Ya make output | Test bloat
⚫ |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
Update tasks templates. Fix bug with token. Check pr template as is.
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
Added github action for validate PR body Added github actions for generate changelog increment
Added github action for validate PR body Added github actions for generate changelog increment
Changelog entry
Added github action for validate PR body
Added github actions for generate changelog increment
Changelog category
Additional information
Added one workflow for validate PR body.
Added two workflows for generate pr to changelog.md - one triggered by cron, another manually.
https://st.yandex-team.ru/KIKIMR-22457
https://st.yandex-team.ru/KIKIMR-22458