Skip to content

Conversation

@mcabbott
Copy link
Member

Closes #574

Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"

[compat]
ChainRulesCore = "1.11.5"
Copy link
Member Author

Choose a reason for hiding this comment

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

Does test/Project.toml need to contain full-time deps again, or are they implied?

Copy link
Member

Choose a reason for hiding this comment

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

They are implied but I would rather not use the test/Project.toml.
It is annoying to keep these things in sync when things move from test only to main dep.

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't mind, should have left it a separate commit... you can fight it out with Seth?

Copy link
Member

Choose a reason for hiding this comment

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

Let's not fight. Simplest to leave it as is then. (I like the convenience of activating the test directory in the REPL while testing stuff)

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

@oxinabox oxinabox left a comment

Choose a reason for hiding this comment

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

Thanks!
I assume you have done a findinfiles VERSION search.
Approved.
subject to reunifying back down to 1 Project.toml.

@mcabbott mcabbott force-pushed the lts16 branch 2 times, most recently from fba4e14 to 9939444 Compare January 19, 2022 15:16
@mcabbott
Copy link
Member Author

mcabbott commented Jan 19, 2022

I assume you have done a findinfiles VERSION search.

Oh no, I guess I searched only src, sorry.

Also there are failures, not sure why. fixed.

@mcabbott mcabbott merged commit 3590f94 into JuliaDiff:main Jan 20, 2022
@mcabbott mcabbott deleted the lts16 branch January 20, 2022 04:04
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.

Drop Julia 1.0?

4 participants