Skip to content

Conversation

jmougeot
Copy link
Contributor

@jmougeot jmougeot commented May 6, 2025

Update the deprecation warning to correctly reference Relation.Binary.Morphism.Bundles instead of Relation.Binary.Reasoning.Morphism.

@jamesmckinna
Copy link
Contributor

As a matter of practice, we don't accept PRs on deprecated modules unless this actually gives rise to a bug...

------------------------------------------------------------------------
-- The Agda standard library
--
-- This module is DEPRECATED. Please use `Relation.Binary.Morphism`
-- instead.
------------------------------------------------------------------------

{-# OPTIONS --cubical-compatible --safe #-}

module Relation.Binary.OrderMorphism where

{-# WARNING_ON_IMPORT
"Relation.Binary.OrderMorphism was deprecated in v1.5.
Use Relation.Binary.Reasoning.Morphism instead."
#-}

What exactly is the bug here?

@JacquesCarette
Copy link
Contributor

The bug is documented in issue #2712 which @jmougeot should have linked.

@jamesmckinna jamesmckinna added this pull request to the merge queue May 7, 2025
Merged via the queue into agda:master with commit 4ee13ab May 7, 2025
2 checks passed
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.

3 participants