-
Notifications
You must be signed in to change notification settings - Fork 197
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
renamed Fibrations.v to HFiber.v #1304
Conversation
@mikeshulman are you able to review this? |
Thanks for this, and sorry for the delay. My main goal here is to make it intuitive which file one needs to import to get a particular result. For instance, it makes sense that to get results about hfibers you need to import |
@mikeshulman What about moving HFiber to Types? Then we will never have to worry about importing it. |
OK, I have rebased and reorganised HFiber. I've moved the "hfiber of functor" results to Types also. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is good, thanks! It might also be worth looking through the library to see if there are some unnecessary imports of HFiber
now that some stuff has been moved into Types
.
They require the fcontr results which rely on Types/Sigma. Maybe we should
kove fcontr to Basics?
…On Wed, 1 Apr 2020 at 13:05, Mike Shulman ***@***.***> wrote:
***@***.**** approved this pull request.
This is good, thanks! It might also be worth looking through the library
to see if there are some unnecessary imports of HFiber now that some
stuff has been moved into Types.
------------------------------
In theories/Types/Equiv.v
<#1304 (comment)>:
> @@ -164,4 +164,25 @@ Section AssumeFunext.
: (A <~> B) <~> (B <~> A)
:= Build_Equiv _ _ ***@***.***_inverse A B) _.
+ (** If [functor_sigma idmap g] is an equivalence then so is g *)
Why are these here rather than in Types/Sigma?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1304 (review)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACBXFE7DEZOK3V2BUD4V5O3RKMUYZANCNFSM4LJP2BOA>
.
|
Ah right. That's probably okay, but maybe we should add a comment in |
Alright before we merge I will have a look for unnecessary imports. |
OK, I've removed the unnecessary imports of HFiber.v |
CI is unhappy? |
The CI is having trouble installing dependencies. I've restarted a few times but still a few (sometimes different) jobs fail. |
So the ubuntu repositories are being hosted on azure. I've heard rumors that azure is under a lot of strain due to current events. It will probably go away tomorrow. |
@mikeshulman Are we good to merge now? |
closes #1303