Skip to content

Реализация игры Монополия + CI/CD и Unit-тесты - #1

Open
Vit0GG wants to merge 12 commits into
code-reviewfrom
main
Open

Vit0GG wants to merge 12 commits into
code-reviewfrom
main

Conversation

@Vit0GG

@Vit0GG Vit0GG commented Mar 5, 2026

Copy link
Copy Markdown
Owner

No description provided.

@Vit0GG
Vit0GG requested a review from gsvgit March 5, 2026 07:13

@gsvgit gsvgit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  1. Не очень понял, зачем в итоге два workflow.
  2. Добавьте в ридми "плашки" со статусом билда, тестов, качества покрытия.

Comment thread .github/workflows/linux.yml Outdated

on:
push:
branches: [ "main", "master" ]

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Вы определитесь, какая у Вас ветка. Но вообще, в случае с домашками, лучше на все.

Comment thread .github/workflows/linux.yml Outdated

jobs:
build-linux:
runs-on: ubuntu-latest

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Договаривались, что ещё, как минимум, винда. Можно ещё и мак.

Comment thread .github/workflows/linux.yml Outdated
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v3

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Кажется, старая версия action. Не перечитывали за нейронкой? Пользоваться нейронками -- ок. Но Вы должны гарантировать, что, во-первых, понимаете, что получилось, во-вторых, что полученный результат качественный.

@gsvgit gsvgit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Нейронка, которой Вы пользуетесь, училась на устаревших данных.

Comment thread .github/workflows/dotnet.yml Outdated

steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Кажется, актуальный не 4-й.

Comment thread .github/workflows/dotnet.yml Outdated
- uses: actions/checkout@v4

- name: Setup .NET
uses: actions/setup-dotnet@v4

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

И тут устаревшая версия.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Мой косяк, мало источников изучил. Получается последние версии
актуальныей setup-dotnet: https://github.com/actions/setup-dotnet/releases
актуальный actions/checkout: https://github.com/actions/checkout/releases/tag/v6.0.2

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.

2 participants