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: broken ci #694

Merged
merged 7 commits into from
Nov 5, 2024
Merged

fix: broken ci #694

merged 7 commits into from
Nov 5, 2024

Conversation

dominikj111
Copy link
Contributor

@dominikj111 dominikj111 commented Oct 25, 2024

The CI is broken due to various issues, in this request all issues with CI are solved.

Unfortunately I had to cherry-pick the commit 6c6399b as it was blockage.

The commands
deno task build
deno fmt --check && deno lint && deno task check:license --check && deno task check:types && deno task test
work well on Deno 1.46.3 as well as on 2.0.2

closes #693

@@ -881,6 +881,9 @@ Deno.test("[e2e] GET /account/upgrade", async (test) => {
await createUser(user);

await test.step("serves internal server error response if the `STRIPE_PREMIUM_PLAN_PRICE_ID` environment variable is not set", async () => {
// Suppress the error message thrown by the handler
const stubbedError = stub(console, "error");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
const stubbedError = stub(console, "error");
using stubbedError = stub(console, "error");

Removes the need for stubbedError.restore(); below.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TS doesn't like it
Screenshot 2024-10-31 at 20 01 45

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah. Ignore then :)

@@ -1,4 +1,7 @@
// Copyright 2023-2024 the Deno authors. All rights reserved. MIT license.

/// <reference lib="deno.unstable" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kt3k shouldn't "unstable": ["kv"] within deno.json fix this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weirdly it didn't. I see there is --unstable-kv option provided when running deno task test, but same problem. I think it could be some deno issue. I have been able to make it work only with the reference ... comment.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, this seems like a runtime issue.

@iuioiua
Copy link
Contributor

iuioiua commented Oct 31, 2024

PTAL @kt3k

Copy link
Member

@kt3k kt3k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for fixing!

@kt3k kt3k changed the title Issue 693 fix broken ci fix: broken ci Nov 5, 2024
@kt3k kt3k merged commit b0b6243 into denoland:main Nov 5, 2024
6 checks passed
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.

Bug report: CI is broken due to missing types
3 participants