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

option to give usage_example in AutoBuild #645

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

option to give usage_example in AutoBuild #645

wants to merge 1 commit into from

Conversation

aminya
Copy link

@aminya aminya commented Jan 30, 2020

No description provided.

@aminya aminya changed the title option to give usage_example option to give usage_example in AutoBuild Jan 30, 2020
Comment on lines 261 to +262
extract_kwargs(kwargs, (:lazy_artifacts,))...,
extract_kwargs(kwargs, (:usage_example,))...,
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps this would be better?

Suggested change
extract_kwargs(kwargs, (:lazy_artifacts,))...,
extract_kwargs(kwargs, (:usage_example,))...,
extract_kwargs(kwargs, (:lazy_artifacts, :usage_example))...,

)
end

function rebuild_jll_packages(name::String, build_version::VersionNumber,
platforms::Vector, products::Vector, dependencies::Vector;
gh_org::String = "JuliaBinaryWrappers",
code_dir::String = joinpath(Pkg.devdir(), "$(name)_jll"),
verbose::Bool = false, lazy_artifacts::Bool = false)
verbose::Bool = false, lazy_artifacts::Bool = false, usage_example::String = default_usage_example)
Copy link
Member

Choose a reason for hiding this comment

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

I am afraid this needs rebasing now

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.

2 participants