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 FillArrays to 1, (keep existing compat) #1380

Merged

Conversation

github-actions[bot]
Copy link
Contributor

This pull request changes the compat entry for the FillArrays package from 0.13.2 to 0.13.2, 1.
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.

@ranocha ranocha force-pushed the compathelper/new_version/2023-03-31-00-05-37-115-01294331447 branch from 76efae3 to 58d3a74 Compare March 31, 2023 00:05
@codecov
Copy link

codecov bot commented Mar 31, 2023

Codecov Report

Merging #1380 (2ab3170) into main (51c7701) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1380   +/-   ##
=======================================
  Coverage   95.96%   95.96%           
=======================================
  Files         344      344           
  Lines       28499    28499           
=======================================
  Hits        27347    27347           
  Misses       1152     1152           
Flag Coverage Δ
unittests 95.96% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@sloede
Copy link
Member

sloede commented Mar 31, 2023

The windows tests seem to fail very consistently now... maybe there is an actual issue?

The failed docs build (https://github.com/trixi-framework/Trixi.jl/actions/runs/4570148496/jobs/8067145007?pr=1380#step:6:1004) is due to #1378

@ranocha
Copy link
Member

ranocha commented Mar 31, 2023

If someone has Windows, it would be great to run tests locally. I suspect it's due to a larger RAM usage of Julia v1.8 compared to v1.7 and maybe reduced RAM available on newer Windows GitHub runers.

Copy link
Member

@ranocha ranocha left a comment

Choose a reason for hiding this comment

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

A discussion on Discourse stated that basically nothing changed, so this should be fine.

@ranocha ranocha enabled auto-merge (squash) April 1, 2023 09:15
@ranocha ranocha disabled auto-merge April 1, 2023 17:33
@ranocha ranocha merged commit dacf28a into main Apr 1, 2023
@ranocha ranocha deleted the compathelper/new_version/2023-03-31-00-05-37-115-01294331447 branch April 1, 2023 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants