Skip to content
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

Adds additional changelog skip (NPFC) & skip checks for nanomap-render #1595

Closed
wants to merge 4 commits into from

Conversation

dj-34
Copy link
Collaborator

@dj-34 dj-34 commented Oct 13, 2024

Что этот PR делает

Closes #1590

Я не даю гарантию что это работает, так как я 0 в Питончике и просто наструячил что-то с помощью GPT.

  • Добавляет возможность скипать валидацию чейнджлога путём удаления cl тегов и добавления слова NPFC (no player facing changes) в текст PR.
  • На Nanomap Render более не будет вешаться лейбл 'CL невалиден'.

Changelog

NPFC

@github-actions github-actions bot added the 🎸 Инструменты Мы выдаем себя за реальное сообщество разработчиков. label Oct 13, 2024
@dj-34
Copy link
Collaborator Author

dj-34 commented Oct 13, 2024

Я не понимаю как мне протестить, я не уверен что в ПРе корректно проходит тест чейнджлога aga

Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts left a comment

Choose a reason for hiding this comment

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

Можно еще тут добавить про NPFC

if len(cl_changes) == 0:
        raise Exception("No changes found in the changelog. Use special label if changelog is not expected.")

tools/changelog/check_changelog.py Outdated Show resolved Hide resolved
tools/changelog/check_changelog.py Show resolved Hide resolved
@dj-34
Copy link
Collaborator Author

dj-34 commented Oct 14, 2024

Помолимся на говнокод и на то, что я вовсе не понимаю что делаю

@dj-34 dj-34 requested a review from m-dzianishchyts October 14, 2024 00:35
@dj-34 dj-34 changed the title Adds skip changelog validation if NPFC present in PR body Adds additional changelog skips & skip for nanomap-render Oct 14, 2024
@dj-34 dj-34 changed the title Adds additional changelog skips & skip for nanomap-render Adds additional changelog skip (NPFC) & skip checks for nanomap-render Oct 14, 2024
@m-dzianishchyts
Copy link
Collaborator

Беру на доработку

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎸 Инструменты Мы выдаем себя за реальное сообщество разработчиков. 📜 CL невалиден
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Убрать проверку на модульность для Nanomaps Render
3 participants