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

Fixed One-to-one property of logs in EL3, EL5, and EL6 #430 #470

Open
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

tdegeorge
Copy link
Contributor

Fixed #430

Copy link

Copy link

Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Copy link

@StevenClontz
Copy link
Contributor

This pull request was not created correctly - it will merge into a feature branch rather than main 😭:

image

@StevenClontz StevenClontz changed the base branch from Tonya-12-16-2024 to main December 18, 2024 00:32
@StevenClontz
Copy link
Contributor

I think this should work now. @AbbyANoble needs to leave an approving review before we can merge.

@StevenClontz StevenClontz added the under-review This PR has been assigned a reviewer label Dec 18, 2024
Copy link

🚀 Preview available 🚀

https://7d91034c.tbil.pages.dev

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why this file is in this pull request? @siwelwerd or @StevenClontz?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think Tonya accidentally branched off of a branch, but I think it's fixed now

@AbbyANoble
Copy link
Contributor

AbbyANoble commented Dec 18, 2024

Ok, I've reviewed it, but at this point I'm lost in the weeds of trying to see the big picture because things weren't compiling due to some missing paragraph tags in remarks. Can someone (@tdegeorge or @siwelwerd) look at what I've done and accept some of these commits? And then not close the issue so we can look it over again once things are compiling?

siwelwerd and others added 11 commits December 19, 2024 15:21
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Co-authored-by: Abby Noble <85640990+AbbyANoble@users.noreply.github.com>
Copy link

🚀 Preview available 🚀

https://5c120d07.tbil.pages.dev

@siwelwerd
Copy link
Contributor

I merged all of your commits @AbbyANoble , but I think it's still a little hard to follow since the remarks aren't showing up right. Maybe @StevenClontz can merge #514 into main to fix those, and then merge main into this so it's easier to see.

@StevenClontz
Copy link
Contributor

Try that...

Copy link

🚀 Preview available 🚀

https://8e81b869.tbil.pages.dev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
precalculus under-review This PR has been assigned a reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

One-to-one property of logs in EL3, EL5, and EL6
4 participants