Skip to content
This repository has been archived by the owner on Aug 28, 2022. It is now read-only.

chore: update rollup/deps and deno version in ci test #33

Closed
wants to merge 3 commits into from
Closed

chore: update rollup/deps and deno version in ci test #33

wants to merge 3 commits into from

Conversation

buttercubz
Copy link
Contributor

No description provided.

@cmorten
Copy link
Owner

cmorten commented May 30, 2021

Hey @buttercubz 👋

Thanks for the contribution! Been on hols without laptop so not been able to whip round and get the latest versions updated in all my modules 😅 thanks for doing some of the chore work!

I run a fmt check in CI, if you run and commit the result of make fmt that should cover off the failures.

@@ -44,7 +44,8 @@ export async function rollup(
};

try {
const bundle = await _rollup(options);
// deno-lint-ignore no-explicit-any
const bundle = await _rollup(options as any);
Copy link
Owner

Choose a reason for hiding this comment

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

Guessing seeing a type mismatch? Check out some of the extended types around https://github.com/cmorten/deno-rollup/blob/main/src/rollup/mod.ts#L165, suspect may have to add the new sanitizeFileName or sim. somewhere, shouldn’t need to cast as any.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i found the problem, now this is fixed

@cmorten cmorten mentioned this pull request May 31, 2021
2 tasks
@cmorten
Copy link
Owner

cmorten commented May 31, 2021

Superseded by #34

@cmorten cmorten closed this May 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants