Skip to content

build: organize tests by goal#17558

Closed
RandyMcMillan wants to merge 2 commits intobitcoin:masterfrom
RandyMcMillan:2019/325/order-by-goal
Closed

build: organize tests by goal#17558
RandyMcMillan wants to merge 2 commits intobitcoin:masterfrom
RandyMcMillan:2019/325/order-by-goal

Conversation

@RandyMcMillan
Copy link
Copy Markdown
Contributor

The tests can be organized by goal for efficiency and clarity.

deploy is before install because this stage is faster
and if it fails - it eliminates the need to continue - which is a
positive for jobs waiting in the cue.

There are other ways to improve efficiency but this one seemed easy
to implement with minimal changes to the .travis.yml configuration.

@fanquake fanquake added the Tests label Nov 22, 2019
@DrahtBot
Copy link
Copy Markdown
Contributor

DrahtBot commented Nov 22, 2019

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #17549 (ci: misc cleanups by MarcoFalke)
  • #17490 (ci: Add valgrind run by MarcoFalke)
  • #17402 (ci: Travis support for PowerPC64 by elichai)
  • #17376 (Add Parallel P2P Client in Rust by TheBlueMatt)
  • #16834 (Fetch Headers over DNS by TheBlueMatt)
  • #16762 (Rust-based Backup over-REST block downloader by TheBlueMatt)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@RandyMcMillan RandyMcMillan deleted the 2019/325/order-by-goal branch November 24, 2019 03:20
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants