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

ci: update test job #2

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

inkydragon
Copy link
Collaborator

@inkydragon inkydragon commented Dec 16, 2024

  • Update test job
  • add dependabot
  • add CompatHelper

@inkydragon
Copy link
Collaborator Author

This repository has been deactivated
Contact an administrator of your git organization to grant write-permissions in your git-provider for this repository.

https://app.codecov.io/github/JuliaPackaging/LazyArtifacts.jl

@inkydragon inkydragon marked this pull request as ready for review December 16, 2024 11:18
@giordano giordano closed this Dec 16, 2024
@giordano giordano reopened this Dec 16, 2024
@giordano
Copy link
Member

Coverage upload failed

@inkydragon
Copy link
Collaborator Author

inkydragon commented Dec 19, 2024

Coverage upload failed

Maybe there is nothing to coverage.
Do we coverage import or using?

module LazyArtifacts
# reexport the Artifacts API
using Artifacts: Artifacts,
artifact_exists, artifact_path, artifact_meta, artifact_hash,
select_downloadable_artifacts, find_artifacts_toml, @artifact_str
export artifact_exists, artifact_path, artifact_meta, artifact_hash,
select_downloadable_artifacts, find_artifacts_toml, @artifact_str
# define a function for satisfying lazy Artifact downloads
using Pkg.Artifacts: ensure_artifact_installed
end

[ Info: CoverageTools.process_folder: Searching src for .jl files...
[ Info: CoverageTools.process_file: Detecting coverage for src/LazyArtifacts.jl
┌ Info: CoverageTools.process_cov: Coverage file(s) for src/LazyArtifacts.jl do not exist.
└ Assuming file has no coverage.

https://github.com/JuliaPackaging/LazyArtifacts.jl/actions/runs/12351907894/job/34664368855?pr=2#step:7:22

@ararslan
Copy link
Member

ararslan commented Jan 8, 2025

Yeah, looks like there's nothing here that we would emit coverage information for. One could equally argue that coverage is 0% or 100%. 😄

Btw, it's probably unnecessary to set up CompatHelper given this has no dependencies aside from stdlibs.

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