Skip to content

Commit

Permalink
sameold,samedec: drop SignificanceLevel::Message
Browse files Browse the repository at this point in the history
The `SignificanceLevel::Message` is not part of NWSI 10-1712 or
any other known standard, and very few event codes in sameold
ever implement it.

Discontinue this enum variant.

The following event codes are affected and are upgraded to
`Statement`s:

* ADR Administrative Message
* NMN Network Message Notification

This is an API-BREAKING change. The samedec CLI is unaffected, but
it is removed from the documentation.
  • Loading branch information
cbs228 committed Feb 23, 2024
1 parent c5292b4 commit 9c0990c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 8 deletions.
1 change: 0 additions & 1 deletion crates/samedec/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -283,7 +283,6 @@ The child process receives the following additional environment variables:
the event code is unknown).

* `T`: Test
* `M`: Message
* `S`: Statement
* `E`: Emergency
* `A`: Watch
Expand Down
7 changes: 0 additions & 7 deletions crates/sameold/src/message/significance.rs
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ use strum::EnumMessage;
/// | Code | Significance |
/// |---------|---------------------------------------------------|
/// | `xxT` | [test](crate::SignificanceLevel::Test) |
/// | `xxM` | [message](crate::SignificanceLevel::Message) |
/// | `xxS` | [statement](crate::SignificanceLevel::Statement) |
/// | `xxE` | [emergency](crate::SignificanceLevel::Emergency) |
/// | `xxA` | [watch](crate::SignificanceLevel::Watch) |
Expand Down Expand Up @@ -80,12 +79,6 @@ pub enum SignificanceLevel {
#[strum(serialize = "T", detailed_message = "Test")]
Test,

/// Message
///
/// A non-emergency message
#[strum(serialize = "M", detailed_message = "Message")]
Message,

/// Statement
///
/// > A message containing follow up information to a warning, watch,
Expand Down

0 comments on commit 9c0990c

Please sign in to comment.