Skip to content

Conversation

@Bownairo
Copy link
Contributor

@Bownairo Bownairo commented Oct 2, 2025

No description provided.

@github-actions github-actions bot added the chore label Oct 2, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request modifies the IC-OS configuration types library (rs/ic_os/config_types/src/lib.rs).

Please ensure you have followed the Configuration Update Protocol guidelines——particularly if adding a new enum or enum variants:

Enum Variant Forward Compatibility Guidelines: If adding a new enum or new variants to an enum, ensure older versions can handle unknown variants gracefully by using #[serde(other)] on a fallback variant. See examples: GuestVMType::Unknown and Ipv6Config::Unknown.

To acknowledge this reminder and unblock the PR, dismiss this code review by:

  • Going to the bottom of the pull request page
  • Finding where this bot is requesting changes
  • Clicking the three dots on the right
  • Selecting "Dismiss review"

For complete guidelines, see the documentation at the top of rs/ic_os/config_types/src/lib.rs.

CONFIG_TYPES_COMPATIBILITY_REMINDER_DEDUP

@Bownairo Bownairo force-pushed the eero/lower-mem branch 3 times, most recently from 783448f to c30e324 Compare October 2, 2025 18:55
@Bownairo Bownairo force-pushed the eero/delete-dev branch 6 times, most recently from 9fb93d6 to 5addf7e Compare October 22, 2025 22:51
Base automatically changed from eero/lower-mem to master October 23, 2025 19:37
@Bownairo Bownairo marked this pull request as ready for review December 8, 2025 22:31
@Bownairo Bownairo requested review from a team as code owners December 8, 2025 22:31
@Bownairo Bownairo changed the title chore: Remove deprecated VM fields chore: [NODE-1683] Remove deprecated VM fields Dec 8, 2025
@Bownairo Bownairo added this pull request to the merge queue Dec 9, 2025
Merged via the queue into master with commit 43c193c Dec 9, 2025
71 of 73 checks passed
@Bownairo Bownairo deleted the eero/delete-dev branch December 9, 2025 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants