Skip to content

Мерджим!#2

Open
dreamIIx wants to merge 89 commits into
masterfrom
kees/dev
Open

Мерджим!#2
dreamIIx wants to merge 89 commits into
masterfrom
kees/dev

Conversation

@dreamIIx

@dreamIIx dreamIIx commented May 6, 2024

Copy link
Copy Markdown
Owner

Чтож... наконец!
Это не стабильная версия, но все же

@dreamIIx dreamIIx added the enhancement New feature or request label May 6, 2024
@dreamIIx dreamIIx requested a review from QQ3646 May 6, 2024 13:40
@dreamIIx dreamIIx self-assigned this May 6, 2024
@dreamIIx dreamIIx added this to the ready for production milestone May 6, 2024
@dreamIIx

dreamIIx commented May 6, 2024

Copy link
Copy Markdown
Owner Author

С другой стороны, думаю нужно доделать 8ю практику сначала, потом убрать ненужный код с мейна и тогда пушить. И проверить билд на обеих платформах. Чтобы собиралось и запускалось гарантированно

@QQ3646

QQ3646 commented May 7, 2024

Copy link
Copy Markdown
Collaborator

Куда жмать, чтобы апрувнуть?

@dreamIIx dreamIIx left a comment

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.

Files changed -> review changes:
Осталось убрать ненужные вещи из мейна, доделать #3 и чекнуть корректность билдов у тебя и у меня. Сейчас у меня все билдается и работает (все прорабатывает, все ловится, таймер в конце тоже)

@dreamIIx dreamIIx requested review from QQ3646 and removed request for QQ3646 May 9, 2024 17:07
Comment thread src/kernel/misc/list.c
return root;
}

node* remove_head(node* root)

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.

impl remove actions with os_delete()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants