appveyor: ignore functional test failure #15376

pull ken2812221 wants to merge 1 commits into bitcoin:master from ken2812221:appveyor-ignore-functional-tests changing 1 files +1 −1
  1. ken2812221 commented at 2:16 PM on February 9, 2019: contributor

    suggested by MarcoFalke in #14446 (comment) Unfortunately that echo doesn't change errorlevel on Windows. So I add a dummy command to set the return code to be 0 at the end.

    Tested in here: https://ci.appveyor.com/project/ken2812221/bitcoin/builds/22245530

  2. appveyor: ignore functional test failure 633fe49e76
  3. ken2812221 commented at 2:16 PM on February 9, 2019: contributor
  4. fanquake added the label Tests on Feb 9, 2019
  5. fanquake requested review from MarcoFalke on Feb 11, 2019
  6. MarcoFalke commented at 1:19 PM on February 11, 2019: member

    It doesn't seem that bad lately. We also have intermittent failures on travis, so I am not sure if this will improve things significantly.

  7. promag commented at 7:19 PM on February 11, 2019: member

    @MarcoFalke said

    but a failing test wouldn't mark a pull request red in the pull request overview tab

    Personally I go and see what failed so I'm fine with having it red.

    I may be wrong but with this change we might end up not taking care of appveyor failures.

  8. ken2812221 commented at 2:17 AM on February 12, 2019: contributor

    Ok. Let's close it.

  9. ken2812221 closed this on Feb 12, 2019

  10. DrahtBot locked this on Dec 16, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-20 09:14 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me