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

Fix "for...of" loop on Result<T> objects #423

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Make `Result<T>` an `IterableIterator`

0963ff5
Select commit
Loading
Failed to load commit list.
Open

Fix "for...of" loop on Result<T> objects #423

Make `Result<T>` an `IterableIterator`
0963ff5
Select commit
Loading
Failed to load commit list.
Travis CI / Travis CI - Pull Request succeeded Dec 25, 2024 in 8m 55s

Build Passed

The build passed, just like the previous build.

Details

This is a pull request build.

It is running a build against the merge commit, after merging #423 Fix "for...of" loop on Result\<T\> objects.
Any changes that have been made to the master branch before the build ran are also included.

Jobs and Stages

This build has seven jobs, running in parallel.

Stage 1: tests

This stage passed.

Job Node.js OS State
3150.1 Run eslint 18 Linux passed
3150.2 TypeScript 4.9 generation and compilation tests 18 Linux passed
3150.3 Unit tests - Node.js 20 20 Linux passed
3150.4 Unit tests w/ latest dependencies - Node.js 20 20 Linux passed
3150.5 Unit tests - Node.js 18 18 Linux passed
3150.6 Unit tests - Node.js 16 16 Linux passed
3150.7 TypeScript (latest) generation and compilation tests 18 Linux passed

Build Configuration

Build Option Setting
Language Node.js
Operating System Linux (Jammy)
Build Configuration
{
  "language": "node_js",
  "os": [
    "linux"
  ],
  "dist": "jammy",
  "sudo": false,
  "cache": {
    "directories": [
      "node_modules"
    ]
  },
  "jobs": {
    "include": [
      {
        "stage": "tests",
        "name": "Run eslint",
        "script": [
          "npm install -g eslint@4; npm run eslint;"
        ],
        "node_js": "18"
      },
      {
        "name": "TypeScript 4.9 generation and compilation tests",
        "node_js": "18",
        "script": [
          "npm install -g [email protected];",
          "pushd test/unit/typescript/",
          "tsc -p .",
          "node -e \"require('./api-generation-test').generate()\" > generated.ts",
          "tsc generated.ts"
        ]
      },
      {
        "name": "Unit tests - Node.js 20",
        "node_js": "20",
        "script": [
          "npm test"
        ]
      },
      {
        "name": "Unit tests w/ latest dependencies - Node.js 20",
        "node_js": "20",
        "script": [
          "rm package-lock.json",
          "rm -rf node_modules",
          "npm install",
          "npm test"
        ]
      },
      {
        "name": "Unit tests - Node.js 18",
        "node_js": "18",
        "script": [
          "npm test"
        ]
      },
      {
        "name": "Unit tests - Node.js 16",
        "node_js": "16",
        "script": [
          "npm test"
        ]
      },
      {
        "name": "TypeScript (latest) generation and compilation tests",
        "node_js": "18",
        "script": [
          "npm install -g typescript;",
          "pushd test/unit/typescript/",
          "tsc -p .",
          "node -e \"require('./api-generation-test').generate()\" > generated.ts",
          "tsc generated.ts"
        ]
      }
    ]
  }
}