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

Do not squash commits #4441

Merged
merged 443 commits into from
Oct 6, 2022
Merged

Do not squash commits #4441

merged 443 commits into from
Oct 6, 2022

Conversation

t-bltg
Copy link
Member

@t-bltg t-bltg commented Oct 6, 2022

I shouldn't have squashed the PR in #4419 (the sanity check does not pass on master πŸ˜“), so here we go again.

using TOML

cd("Plots.jl")

for pkg ∈ ("RecipesBase", "RecipesPipeline", "Plots")
  versions = TOML.parsefile(joinpath(first(DEPOT_PATH), "registries", "General", pkg[1:1], pkg, "Versions.toml"))

  found = all(sort(collect(keys(versions)))) do version
    tree_sha = versions[version]["git-tree-sha1"]
    try
      readchomp(`git rev-parse -q --verify "$(tree_sha)^{tree}"`)
      return true
    catch
      return false
    end
  end
  println(pkg, ": ", found ? "all versions found!" : "ney")
end

t-bltg and others added 26 commits July 9, 2022 21:51
* update compat bounds

* run snoop on master only

* update precompile
Co-authored-by: t-bltg <t-bltg@users.noreply.github.com>
Co-authored-by: t-bltg <t-bltg@users.noreply.github.com>
Co-authored-by: BeastyBlacksmith <BeastyBlacksmith@users.noreply.github.com>
* document `@layout`

* rewok precompile statements using `SnoopPrecompile`

* add ci

* bump julia to `1.6` for failing CI
@t-bltg t-bltg merged commit 68fda3e into JuliaPlots:master Oct 6, 2022
@t-bltg t-bltg deleted the squash branch October 6, 2022 13:06
@t-bltg
Copy link
Member Author

t-bltg commented Oct 6, 2022

Post-merge comment: sanity check works on master.

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.