Skip to content

CompatHelper: add new compat entry for AbstractDifferentiation at version 0.5 for package docs, (keep existing compat)#112

Merged
devmotion merged 1 commit intomasterfrom
compathelper/new_version/2023-09-26-00-31-43-507-02041740450
Oct 5, 2023
Merged

CompatHelper: add new compat entry for AbstractDifferentiation at version 0.5 for package docs, (keep existing compat)#112
devmotion merged 1 commit intomasterfrom
compathelper/new_version/2023-09-26-00-31-43-507-02041740450

Conversation

@github-actions
Copy link
Contributor

This pull request sets the compat entry for the AbstractDifferentiation package to 0.5 for package docs.
This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.
Note: Consider registering a new release of your package immediately after merging this PR, as downstream packages may depend on this for tests to pass.

…sion 0.5 for package docs, (keep existing compat)
@devmotion
Copy link
Member

IMO (as discussed in #101) we should not fix the version of AbstractDifferentiation in docs/Project.toml since this will be problematic when working on breaking releases. Rather, I'd suggest removing AbstractDifferentiation from the deps section - the first thing that's done on Github CI or when building locally, is adding the dev package at the repo anyway.

@gdalle
Copy link
Member

gdalle commented Oct 5, 2023

Rather, I'd suggest removing AbstractDifferentiation from the deps section - the first thing that's done on Github CI or when building locally, is adding the dev package at the repo anyway.

The reason I disagree is that when you build locally, dev-ing the package changes docs/Project.toml to add the line you want to remove. Do you think there is a way around that?

@devmotion
Copy link
Member

No, I don't know any. As long as you don't have any other changes in the Project.toml file locally, it's not a problem - you just should not commit them. I tend to think that the many annoyances with CompatHelper and breaking changes in CI outweigh the inconveniences the few times you actually build docs locally (I almost always only use CI).

@gdalle
Copy link
Member

gdalle commented Oct 5, 2023

(I almost always only use CI).

I guess that's the root of our disagreement then, I always build locally ;)

Also note that the CI preview of the docs is only available for PRs from repo branches, and not for PRs from forks.
That means outside contributors are forced to build locally, and they are also more likely to commit the changes to docs/Project.toml by mistake.

@devmotion
Copy link
Member

they are also more likely to commit the changes to docs/Project.toml by mistake.

It's easy to notice it and tell them to revert the change when doing the PR review 🤷

@devmotion
Copy link
Member

Since you build locally and it would be less convenient for you with the approach I commonly use, let's just merge this PR and re-evaluate if it becomes too inconvenient/problematic for CI.

@devmotion devmotion merged commit 0559f3b into master Oct 5, 2023
@devmotion devmotion deleted the compathelper/new_version/2023-09-26-00-31-43-507-02041740450 branch October 5, 2023 21:36
@devmotion
Copy link
Member

As mentioned above, the compat entry is problematic when working on breaking releases: https://github.com/JuliaDiff/AbstractDifferentiation.jl/actions/runs/6424726444/job/17445945926#step:4:18

@devmotion devmotion mentioned this pull request Oct 5, 2023
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.

2 participants