Skip to content

Revert "[node] Fix http.createServer signature"#70453

Merged
andrewbranch merged 1 commit intomasterfrom
revert-70289-fix/http-prototypes
Sep 4, 2024
Merged

Revert "[node] Fix http.createServer signature"#70453
andrewbranch merged 1 commit intomasterfrom
revert-70289-fix/http-prototypes

Conversation

@andrewbranch
Copy link
Member

Reverts #70289

@typescript-bot
Copy link
Contributor

typescript-bot commented Sep 4, 2024

@andrewbranch Thank you for submitting this PR!

This is a live comment that I will keep updated.

3 packages in this PR

Code Reviews

Because this is a widely-used package, a DT maintainer will need to review it before it can be merged.

You can test the changes of this PR in the Playground.

Status

  • ✅ No merge conflicts
  • ❌ Continuous integration tests have failed
  • 🕐 A DT maintainer needs to approve changes that affect more than one package

Once every item on this list is checked, I'll ask you for permission to merge and publish the changes.


Diagnostic Information: What the bot saw about this PR
{
  "type": "info",
  "now": "-",
  "pr_number": 70453,
  "author": "andrewbranch",
  "headCommitOid": "0f6d3a030264809f97f85f897481fa102901634c",
  "mergeBaseOid": "34dabbdd4e64d72c09230353fa8b9868d0735f2d",
  "lastPushDate": "2024-09-04T20:43:23.000Z",
  "lastActivityDate": "2024-09-04T20:43:23.000Z",
  "hasMergeConflict": false,
  "isFirstContribution": false,
  "tooManyFiles": false,
  "hugeChange": false,
  "popularityLevel": "Critical",
  "pkgInfo": [
    {
      "name": "node-red__registry",
      "kind": "edit",
      "files": [
        {
          "path": "types/node-red__registry/node-red__registry-tests.ts",
          "kind": "test"
        }
      ],
      "owners": [
        "alexk111",
        "Shaquu"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Well-liked by everyone"
    },
    {
      "name": "node",
      "kind": "edit",
      "files": [
        {
          "path": "types/node/http.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/http2.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/https.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/test/http.ts",
          "kind": "test"
        },
        {
          "path": "types/node/test/http2.ts",
          "kind": "test"
        },
        {
          "path": "types/node/v16/http.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/v16/http2.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/v16/https.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/v16/test/http2.ts",
          "kind": "test"
        },
        {
          "path": "types/node/v18/http.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/v18/http2.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/v18/https.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/v18/test/http2.ts",
          "kind": "test"
        },
        {
          "path": "types/node/v20/http.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/v20/http2.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/v20/https.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/node/v20/test/http2.ts",
          "kind": "test"
        }
      ],
      "owners": [
        "Microsoft",
        "jkomyno",
        "alvis",
        "r3nya",
        "btoueg",
        "smac89",
        "touffy",
        "DeividasBakanas",
        "eyqs",
        "Hannes-Magnusson-CK",
        "hoo29",
        "kjin",
        "ajafff",
        "islishude",
        "mwiktorczyk",
        "mohsen1",
        "galkin",
        "parambirs",
        "eps1lon",
        "ThomasdenH",
        "WilcoBakker",
        "wwwy3y3",
        "samuela",
        "kuehlein",
        "bhongy",
        "chyzwar",
        "trivikr",
        "yoursunny",
        "qwelias",
        "ExE-Boss",
        "peterblazejewicz",
        "addaleax",
        "victorperin",
        "ZYSzys",
        "NodeJS",
        "LinusU",
        "wafuwafu13",
        "mcollina",
        "Semigradsky"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Critical"
    },
    {
      "name": "stoppable",
      "kind": "edit",
      "files": [
        {
          "path": "types/stoppable/stoppable-tests.ts",
          "kind": "test"
        }
      ],
      "owners": [
        "EricByers",
        "jplusje",
        "BendingBender"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Popular"
    }
  ],
  "reviews": [],
  "mainBotCommentID": 2329955327,
  "ciResult": "fail",
  "ciUrl": "https://github.com/DefinitelyTyped/DefinitelyTyped/commit/0f6d3a030264809f97f85f897481fa102901634c/checks?check_suite_id=27991649595"
}

@typescript-bot typescript-bot added the The CI failed When GH Actions fails label Sep 4, 2024
@typescript-bot
Copy link
Contributor

@andrewbranch The CI build failed! Please review the logs for more information.

Once you've pushed the fixes, the build will automatically re-run. Thanks!

Note: builds that are failing do not end up on the list of PRs for the DT maintainers to review.

@andrewbranch andrewbranch merged commit a75e4ca into master Sep 4, 2024
@andrewbranch andrewbranch deleted the revert-70289-fix/http-prototypes branch September 4, 2024 21:18
daelmaak pushed a commit to daelmaak/DefinitelyTyped that referenced this pull request Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants