Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate inspect function in favour of new with ... in construct #1580

Open
MatthewDaggitt opened this issue Aug 19, 2021 · 6 comments
Open
Labels
deprecation refactoring status: blocked-by-issue Progress on this issue or PR is blocked by another issue. upstream Changes induced by Agda upstream
Milestone

Comments

@MatthewDaggitt
Copy link
Contributor

@gallais added the inspect idiom as a first class language construct in Agda 2.6.2. We should consider using it in the library to simplify our code, and possibly deprecating the old inspect function to encourage other people to migrate as well.

@ajrouvoet
Copy link
Contributor

I hope this would also (eventually) get rid of the imported [_] from PropositionalEquality which clashes with list singletons and others.

@MatthewDaggitt
Copy link
Contributor Author

You're not the only one who has expressed this wish, @jamesmckinna also mentioned this. It will be deprecated first, but yes eventually it'll be removed.

@jamesmckinna
Copy link
Contributor

jamesmckinna commented Aug 10, 2023

Suggest closing this now, after the merge of #1930, notwithstanding the issues of strictness of inspect vs. laziness of with ... in ... raised by agda#5833

@jamesmckinna jamesmckinna modified the milestones: Agda-future, v2.0 Aug 10, 2023
@jamesmckinna
Copy link
Contributor

jamesmckinna commented Sep 13, 2023

The saga continues,unfortunately: agda/agda#6841 so, no, we should probably keep this open,even though the stdlib uses of inspect have been successfully massaged away, the deprecation itself might have to be reverted for the time being... sigh.

@jamesmckinna
Copy link
Contributor

For v2.0: suggest removing the deprecation warning?

@MatthewDaggitt MatthewDaggitt added status: blocked-by-issue Progress on this issue or PR is blocked by another issue. upstream Changes induced by Agda upstream labels Sep 27, 2023
@MatthewDaggitt MatthewDaggitt modified the milestones: v2.0, Agda-future Sep 27, 2023
@MatthewDaggitt
Copy link
Contributor Author

Opened a PR undeprecating it, and have moved this to Agda-future

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deprecation refactoring status: blocked-by-issue Progress on this issue or PR is blocked by another issue. upstream Changes induced by Agda upstream
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants