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

Добавил вторую часть Тестирования #36

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

lgnv
Copy link
Collaborator

@lgnv lgnv commented Dec 7, 2024

No description provided.


namespace Advanced.Classwork.ThingCache;

public class ThingCache
Copy link

Choose a reason for hiding this comment

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

А не хочешь унаследовать от IThingService?
Или думаешь, что это просто лишних проблем создаст?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

можно, но может запутать, в целом код взят из прошлой версии тестирования, поэтому оставлю

@@ -0,0 +1,37 @@
# Тестирование. Часть 2
Copy link

Choose a reason for hiding this comment

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

А после этого есть домашка? Прикрепишь ее?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

домашки нет

push:
branches: [ "master" ]
pull_request:
branches: [ "master" ]
Copy link

Choose a reason for hiding this comment

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

А поч только для мастера?
Тогда ребята не потыкаются в него. Так можно сделать их жизнь повеселее)

Copy link
Collaborator Author

@lgnv lgnv Dec 10, 2024

Choose a reason for hiding this comment

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

в этом году они в любом случае не потыкаются, так как домашки нет -- только на паре расскажу

а в целом, как я понял, этот фильтр сработает если есть есть мастер в into или в from ветках
так например, в текущем PR прогон дернулся
image

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