Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

CompatHelper: bump compat for "Reexport" to "1.0" #544

Merged

Conversation

github-actions[bot]
Copy link
Contributor

This pull request changes the compat entry for the Reexport package from 0.2 to 0.2, 1.0.

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.

@timholy
Copy link
Member

timholy commented Dec 26, 2020

Weird that the CI workflow didn't run.

@giordano
Copy link
Contributor

It isn't weird: actions don't trigger other actions by default, unless you give them access to private keys. This is rather frustrating. Closing/reopening the PR should work though

@timholy
Copy link
Member

timholy commented Dec 26, 2020

Ah, duh, right. Would be easy to set up the keys.

@timholy timholy closed this Dec 26, 2020
@timholy timholy reopened this Dec 26, 2020
@codecov
Copy link

codecov bot commented Dec 26, 2020

Codecov Report

Merging #544 (853e3f1) into master (9767ac1) will decrease coverage by 2.58%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #544      +/-   ##
==========================================
- Coverage   49.94%   47.35%   -2.59%     
==========================================
  Files          32       32              
  Lines        2515     2365     -150     
==========================================
- Hits         1256     1120     -136     
+ Misses       1259     1245      -14     
Impacted Files Coverage Δ
src/GLib/gerror.jl 33.33% <0.00%> (-6.67%) ⬇️
src/cairo.jl 70.00% <0.00%> (-4.29%) ⬇️
src/lists.jl 55.28% <0.00%> (-4.08%) ⬇️
src/text.jl 38.09% <0.00%> (-3.83%) ⬇️
src/layout.jl 23.86% <0.00%> (-3.32%) ⬇️
src/container.jl 44.44% <0.00%> (-3.06%) ⬇️
src/toolbar.jl 75.00% <0.00%> (-2.78%) ⬇️
src/buttons.jl 62.66% <0.00%> (-2.77%) ⬇️
src/theme.jl 72.72% <0.00%> (-2.28%) ⬇️
src/selectors.jl 21.35% <0.00%> (-2.23%) ⬇️
... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9767ac1...853e3f1. Read the comment docs.

Copy link
Member

@timholy timholy left a comment

Choose a reason for hiding this comment

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

I've verified that the tests indeed ran with Reexport 1.0, so this looks good to merge. Would also be nice to tag a new release, if there aren't other issues @tknopp.

@tknopp tknopp merged commit 9035f82 into master Dec 30, 2020
@tknopp tknopp deleted the compathelper/new_version/2020-12-26-00-32-14-898-588244134 branch December 30, 2020 21:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants