Add a saturating `adjust` variant?
還沒有人認領這個 Issue。
評估
- 難度
- 5/5
- 預估耗時
- 一週以上
- 新手友好度
- 25/100
- Issue 類型
- 功能
- 描述清晰度
- 基本清楚
- 活躍度
- 停滯
- 領域
- backend
研究方向
The issue proposes a saturating shift variant of adjust in the DateTime library. Start by reading the existing adjust implementation and the Duration type; decide whether saturating behavior is appropriate and how bounds should be represented. Done means the API design is agreed on and the requested variant is specified.
由索引模型根據 Issue 內容生成。
描述
While writing an app with lots of DateTime manipulations, I noticed that adjust is very annoying to use, because of the Maybe return type. I don't think this is a very practical design, similarly to how (+) :: Int -> Int -> Maybe Int wouldn't be. There is often not much you can do on failure, other than just skip the operation. Needless to say, that doesn't help with correctness.
I think that saturating semantics would make more sense. Something like this:
-- | Like `adjust`, but saturates on out-of-bounds results.
shift :: forall d. Duration d => d -> DateTime -> DateTime
shift d dt = case adjust d dt of
Nothing -> if fromDuration d > mempty then top else bottom
Just d' -> d'
Does this sound like a good idea? I can make it into a PR, but wanted to get some feedback first.
- 主要語言
- PureScript
- 星號
- 51
- 分支
- 48
- 平均合併
- 2 天 23 小時
- 30 天內合併 PR
- 1
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 有 Pull Request 範本
- 沒有貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
purescript/purescript-datetime 的其他 Issue
-
難度 3/5 1-2 天 新手友好度 25/100
purescript/purescript-datetime#102 · 5 則留言 ·
-
Add constructors for Date / Time components using type-level integers可能重新可做 關聯的 PR 已關閉且未合併。 未關閉
難度 5/5 一週以上 新手友好度 15/100
purescript/purescript-datetime#96 · 1 則留言 ·
-
type: documentation
難度 2/5 1-3 小時 新手友好度 35/100
purescript/purescript-datetime#92 · 1 則留言 ·
-
status: needs more info
難度 5/5 一週以上 新手友好度 20/100
-
type: enhancement
難度 5/5 一週以上 新手友好度 24/100
purescript/purescript-datetime#79 · 6 則留言 · 1 個 reaction ·
查看 purescript/purescript-datetime 的全部 Issue
相似的 Issue
-
ManageSieve proxy: a backend's NO/BYE greeting is skipped as a capability line and reported as EOF未關閉bug
難度 2/5 1-3 小時 新手友好度 72/100
維護者通常 1 天內回覆
-
Mend: dependency security vulnerability
難度 2/5 1-3 小時 新手友好度 62/100
ManageIQ/manageiq-pods#1476 ·
維護者通常 1 天內回覆
-
mcp: typed tool handlers lose wrapped JSON-RPC errors可能重新可做 關聯的 PR 已關閉且未合併。 未關閉
難度 2/5 1-3 小時 新手友好度 85/100
modelcontextprotocol/go-sdk#1529 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 66/100
FinanceFlash/unvibecode#818 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 70/100