Remove Deno shims

[PonomareVlad]

Proposed to remove Deno shims — grammyjs/conversations#82 (comment)

Test source

Closes #7

[wojpawlik]

Closes #7.

@KnorpelSenf what's the lightweight tag for?

oson/build.ts

Lines 47 to 50 in 0fe5862

	const process = Deno.run({
	cmd: ["git", "tag", version],
	});
	await process.status();

[KnorpelSenf]

I don't see why we should take web-compatible code, inject a bunch of weird things so it can run on Node, and then use the Node-compat mode of web runtimes?

[KnorpelSenf]

But then again, I care so little about what happens to the Node package, that I might as well merge. As long as the web-compatible Deno build is clean, I'm happy. Sorry, future self :)

[KnorpelSenf]

Closes #7.

@KnorpelSenf what's the lightweight tag for?

oson/build.ts

Lines 47 to 50 in 0fe5862

	const process = Deno.run({
	cmd: ["git", "tag", version],
	});
	await process.status();

It tags a new release

[PonomareVlad]

@KnorpelSenf This will happen.

Just exclude test sources (and polyfills for them) from npm package and it will be look cleaner 🙂

[codecov-commenter]

Codecov Report

❗ No coverage uploaded for pull request base (main@0fe5862). Click here to learn what that means.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage        ?   95.11%           
=======================================
  Files           ?        3           
  Lines           ?      307           
  Branches        ?       68           
=======================================
  Hits            ?      292           
  Misses          ?       14           
  Partials        ?        1           

[KnorpelSenf]

@KnorpelSenf This will happen

What do you mean?

[PonomareVlad]

@KnorpelSenf This will happen

What do you mean?

Just exclude test sources (and polyfills for them) from npm package and it will be look cleaner 🙂

[KnorpelSenf]

In package.json?

Do you mean I should do it? Do you mean this PQ already does it? Do you mean that this would be something to solve in the future? Are you responding to anything I said? I can't quite follow tbh

[PonomareVlad]

@KnorpelSenf I just woke up, I'm sorry, caffeine will work now and I'll add it to PQ 🙃

[PonomareVlad]

@KnorpelSenf If we disable test then got this result

This seems like simplest way to eliminate test sources (and shims for them), but if test step is important, then we need something else

Maybe we can run build in two iterations — first with test and second without it for minimize ?

[wojpawlik]

https://github.com/denoland/dnt#denotest-only-shim

@KnorpelSenf annotated tags are supposed to be used fot releases

[KnightNiwrem]

Ref: https://t.me/grammyjs/155170

We cannot release a change that would break oson for all non-VercelEdge Node users.

Edit: Those classes have been made available in Node's global

[PonomareVlad]

Maybe we need to pin minimal node version for reliability ?

[KnorpelSenf]

I had to do some adjustments which were partially included here. Could you resolve the conflicts? I'll get back to this after that.