Skip to content

refactor: pin dependencies#49

Merged
AsakuraMizu merged 16 commits intomainfrom
refactor
Nov 25, 2025
Merged

refactor: pin dependencies#49
AsakuraMizu merged 16 commits intomainfrom
refactor

Conversation

@AsakuraMizu
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings November 23, 2025 14:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors error handling by replacing AxError::Other(LinuxError::*) with AxError::from(LinuxError::*) across the codebase, updates dependency versions (axerrno to 0.2, axio to 0.2), and removes crates-io patches in favor of direct dependencies. Additionally, it updates a console API call from irq_number() to irq_num() and removes an unnecessary type cast.

  • Standardizes error conversion using From trait instead of Other variant
  • Updates dependencies to version 0.2 and removes patch overrides
  • Refactors console IRQ API usage

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
arceos Updates subproject commit reference
api/src/vfs/dev/tty/ntty.rs Updates console IRQ function name and removes type cast
api/src/vfs/dev/loop.rs Converts error handling to use From trait
api/src/syscall/task/wait.rs Converts error handling to use From trait
api/src/syscall/sync/futex.rs Converts error handling to use From trait
api/src/syscall/net/socket.rs Converts error handling to use From trait
api/src/syscall/net/opt.rs Converts error handling to use From trait
api/src/socket.rs Converts error handling to use From trait
Cargo.toml Updates dependency versions and removes patch section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AsakuraMizu AsakuraMizu requested a review from Copilot November 25, 2025 13:28
@AsakuraMizu AsakuraMizu marked this pull request as ready for review November 25, 2025 13:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AsakuraMizu AsakuraMizu requested review from Azure-stars and removed request for Azure-stars November 25, 2025 14:04
@AsakuraMizu AsakuraMizu merged commit 7281f14 into main Nov 25, 2025
12 checks passed
@AsakuraMizu AsakuraMizu deleted the refactor branch November 25, 2025 15:16
@AsakuraMizu AsakuraMizu changed the title wip: refactor refactor: pin dependencies Nov 27, 2025
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