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

chore(*): remove is_absolute_value #16340

Closed
wants to merge 7 commits into from
Closed

chore(*): remove is_absolute_value #16340

wants to merge 7 commits into from

Conversation

ericrbg
Copy link
Collaborator

@ericrbg ericrbg commented Sep 1, 2022

This is still a WIP - thoughts very welcome!


Open in Gitpod

@ericrbg ericrbg added RFC Request for comment WIP Work in progress labels Sep 1, 2022
@YaelDillies
Copy link
Collaborator

The diff looks great!

I suggest letting #16227 through first, because you're introducing a lot of uses of dot notation here and you won't need any of them after #16227.

@mathlib-dependent-issues-bot mathlib-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Sep 1, 2022
bors bot pushed a commit that referenced this pull request Sep 24, 2022
Doing this this way round makes the refactor in #16340 much easier. I will follow up this PR with similar PRs for `padic_norm` and `padic_norm_e`.



Co-authored-by: Eric Rodriguez <37984851+ericrbg@users.noreply.github.com>
@mathlib-dependent-issues-bot mathlib-dependent-issues-bot removed the blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. label Sep 24, 2022
@mathlib-dependent-issues-bot
Copy link
Collaborator

This PR/issue depends on:

bottine pushed a commit that referenced this pull request Sep 25, 2022
Doing this this way round makes the refactor in #16340 much easier. I will follow up this PR with similar PRs for `padic_norm` and `padic_norm_e`.



Co-authored-by: Eric Rodriguez <37984851+ericrbg@users.noreply.github.com>
@ericrbg ericrbg closed this Apr 22, 2023
@ericrbg ericrbg deleted the bundle_abv branch April 22, 2023 21:05
@YaelDillies
Copy link
Collaborator

To be redone after the port, I assume?

@ericrbg
Copy link
Collaborator Author

ericrbg commented Apr 22, 2023

If I have time :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maybe-later RFC Request for comment WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants