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

CompatHelper: bump compat for Functors to 0.5, (keep existing compat) #344

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Nov 5, 2024

This pull request changes the compat entry for the Functors package from 0.1, 0.2, 0.3, 0.4 to 0.1, 0.2, 0.3, 0.4, 0.5.
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.

@yebai yebai closed this Nov 6, 2024
@yebai yebai reopened this Nov 6, 2024
@yebai
Copy link
Member

yebai commented Nov 6, 2024

@willtebbutt
Copy link
Member

@penelopeysm any thoughts on this? It looks like version 0.4.36 is installed, for which isdefined(Mooncake, :MooncakeRuleCompilationError) returns true, so I would have thought that this line would be hit, but it looks like the line two lines below it has been hit.

@penelopeysm
Copy link
Member

I think this was fixed in #343

@penelopeysm
Copy link
Member

I mean, not the fact that it throws an error, but the check in CI was updated to catch the right error :')

@penelopeysm
Copy link
Member

Remaining test failures are due to Enzyme, so I think this is safe to merge.

@yebai yebai merged commit e99db76 into master Nov 6, 2024
23 of 27 checks passed
@yebai yebai deleted the compathelper/new_version/2024-11-05-00-23-01-178-00381676774 branch November 6, 2024 14:38
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.

3 participants