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

Add optional LTS build - runs after all others, not required to pass. #5347

Merged
merged 1 commit into from
Jan 30, 2019

Conversation

danpat
Copy link
Member

@danpat danpat commented Jan 28, 2019

Issue

Adds an optional build against Node LTS (whatever version that is at the time). Will run after all other builds, and is not required for a build to pass.

Gives us an opportunity to canary upcoming LTS releases without needing to make sure they pass.

Note: this breaks the build into stages. It's possible that Node 10 and LTS are the same thing, so we don't want the LTS build to cause the Node 10 build to fail by publishing binaries first (node-pre-gyp will error if binaries are already uploaded).

Tasklist

Requirements / Relations

Link any requirements here. Other pull requests this PR is based on?

…ive us a heads up when new Node

versions break bindings, and might automagically get us support for new versions if it "just works".
@danpat danpat force-pushed the danpat_optional_lts_build branch from e8d7034 to eaf8b89 Compare January 30, 2019 20:16
@danpat danpat merged commit 4c665b2 into master Jan 30, 2019
datendelphin added a commit to fossgis-routing-server/osrm-backend that referenced this pull request Nov 19, 2020
  - Changes from 5.21.0
    - Build:
      - ADDED: optionally build Node `lts` and `latest` bindings [Project-OSRM#5347](Project-OSRM#5347)
    - Features:
      - ADDED: new waypoints parameter to the `route` plugin, enabling silent waypoints [Project-OSRM#5345](Project-OSRM#5345)
      - ADDED: data timestamp information in the response (saved in new file `.osrm.timestamp`). [Project-OSRM#5115](Project-OSRM#5115)
@DennisOSRM DennisOSRM deleted the danpat_optional_lts_build branch November 6, 2022 14:12
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.

1 participant