search

boostorg / algorithm

Boost.org algorithm module
http://boost.org/libs/algorithm
Boost Software License 1.0
114 stars 105 forks source link

Enhance CI, add license and readme, and fix some issues #56

Closed jeking3 closed 2 years ago

jeking3 commented 6 years ago

This fixes #52

Note the UBSAN job is expected to fail, it needs to be investigated and resolved. I added symbol visibility to the exceptions but that didn't make a difference (it should be there anyway).

Since you do not have Travis CI or AppVeyor enabled on this repository, here are the build results:

AppVeyor: https://ci.appveyor.com/project/jeking3/algorithm/builds/20008102 Travis CI: https://travis-ci.org/jeking3/algorithm/builds/449921988

jeking3 commented 6 years ago

@mclow a few things to do here:

  1. Enable Travis CI builds on boostorg/algorithm
  2. Enable AppVeyor builds on boostorg/algorithm
  3. Analyze the UBSAN in the travis result link in the description. It's a bad vtable in a dtor.

As admin for this repo you must do items 1 and 2 to enable PRs and merges into develop and master to run a build.

jeking3 commented 5 years ago

Wow did you pay for travis-ci.com?

codecov[bot] commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@9477cd8). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop      #56   +/-   ##
==========================================
  Coverage           ?   80.75%           
==========================================
  Files              ?       39           
  Lines              ?      717           
  Branches           ?      236           
==========================================
  Hits               ?      579           
  Misses             ?       31           
  Partials           ?      107
Impacted Files Coverage Δ
include/boost/algorithm/apply_permutation.hpp 100% <ø> (ø)
include/boost/algorithm/hex.hpp 75.36% <ø> (ø)

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 9477cd8...ac1713e. Read the comment docs.

jeking3 commented 5 years ago

This was passing on my fork... https://github.com/jeking3/algorithm/pull/2

jgopel commented 2 years ago

What's the status on this? I think that landing a change along these lines would help me a lot, as I could offload most of my b2 permutations to CI.