Skip to content

ci: standardise workflows using SciML's reusable workflows#170

Merged
thazhemadam merged 5 commits intomainfrom
at/use-reusable-workflows
Jul 29, 2024
Merged

ci: standardise workflows using SciML's reusable workflows#170
thazhemadam merged 5 commits intomainfrom
at/use-reusable-workflows

Conversation

@thazhemadam
Copy link
Member

Update the workflows in this repository to use SciML's reusable workflows.
This is part of a larger effort to standardise the SciML's CI workflows for more generic and common requirements, to keep the workflows uniform and easier to maintain.

@thazhemadam thazhemadam force-pushed the at/use-reusable-workflows branch from d14b928 to 4e3d944 Compare July 29, 2024 10:17
All the deprecations in the package itself have been appropriately
fixed up.
However, currently, tests fail when `depwarn` is set to error because of
a transitive dependency's depwarn, which is not directly influenced by
the package's primitives itself.

```julia
WARNING: using deprecated binding Colors.RGB1 in PlotUtils.
, use XRGB instead.
PlotUtils.RGB1 is deprecated, use ColorTypes.XRGB{T} where T<:Union{AbstractFloat, FixedPointNumbers.FixedPoint{T, f} where f where T<:Integer} instead.

WARNING: using deprecated binding Colors.RGB4 in PlotUtils.
, use RGBX instead.
PlotUtils.RGB4 is deprecated, use ColorTypes.RGBX{T} where T<:Union{AbstractFloat, FixedPointNumbers.FixedPoint{T, f} where f where T<:Integer} instead.
```

Thus, set `depwarn` to just `yes` while runnning the julia tests, so
that CI can pass.
@thazhemadam thazhemadam merged commit 3f1d614 into main Jul 29, 2024
@thazhemadam thazhemadam deleted the at/use-reusable-workflows branch July 29, 2024 11:50
@ranocha
Copy link
Member

ranocha commented Jul 29, 2024

@thazhemadam No coverage has been uploaded after merging this PR. This isn't very pleasant since I would like to keep track of coverage information with both Coveralls and Codecov.

@ChrisRackauckas
Copy link
Member

We can discuss this at the top level in https://github.com/SciML/.github. Codecov on forks is broken though at the Codecov service level, so we may just need to globally move everything to coveralls.

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