fix: introduce alternate attestation API implementation (resolves #28) #35

Open

gitlimes wants to merge 1 commits from gitlimes/main into main

gitlimes commented

2023-07-19 23:15:52 +02:00

(Migrated from github.com)

There's little I could say that probably hasn't already been said by other people in #28.

So here's something that definitely hasn't been said before:

hewwo googwe~
if it can math, i can twust~
so couwd you pweatty pwease not fuck us ovew wif youw buwwshit fow owonce? >:3
thanks :3

There's little I could say that probably hasn't already been said by other people in #28. So here's something that definitely hasn't been said before: hewwo googwe~ if it can math, i can twust~ so couwd you pweatty pwease not fuck us ovew wif youw buwwshit fow owonce? >:3 thanks :3

👍 5 ❤️ 3 🎉 3 🚀 3

maj113 (Migrated from github.com) approved these changes 2023-07-19 23:39:30 +02:00

Tau5 (Migrated from github.com) approved these changes 2023-07-20 00:05:13 +02:00

mokrates (Migrated from github.com) approved these changes 2023-07-20 00:10:00 +02:00

mokrates (Migrated from github.com) left a comment

LGTM

Jack5079 (Migrated from github.com) approved these changes 2023-07-20 01:05:42 +02:00

adryzz (Migrated from github.com) approved these changes 2023-07-20 03:11:39 +02:00

joebobbio (Migrated from github.com) approved these changes 2023-07-20 03:24:38 +02:00

joebobbio (Migrated from github.com) left a comment

lgtm

perillamint (Migrated from github.com) approved these changes 2023-07-20 04:13:29 +02:00

perillamint (Migrated from github.com) left a comment

LGTM

natep-tech (Migrated from github.com) approved these changes 2023-07-20 06:21:26 +02:00

SteffoSpieler (Migrated from github.com) approved these changes 2023-07-20 12:37:46 +02:00

SteffoSpieler (Migrated from github.com) left a comment

lgtm

chfour (Migrated from github.com) approved these changes 2023-07-20 19:27:37 +02:00

chfour (Migrated from github.com) left a comment

lgtm but minor nitpick: maybe we could just reduce it to a const attestation = () => 1 + 1 === 2? no need for the braces

lgtm but minor nitpick: maybe we could just reduce it to a `const attestation = () => 1 + 1 === 2`? no need for the braces

gitlimes commented

2023-07-20 23:47:03 +02:00

(Migrated from github.com)

lgtm but minor nitpick: maybe we could just reduce it to a const attestation = () => 1 + 1 === 2? no need for the braces

that's a good idea! personally, i like using the curly braces for clarity, but we can always just let the website choose the preferred form, considering how little of a size difference there is between implementations

> lgtm but minor nitpick: maybe we could just reduce it to a `const attestation = () => 1 + 1 === 2`? no need for the braces that's a good idea! personally, i like using the curly braces for clarity, but we can always just let the website choose the preferred form, considering how little of a size difference there is between implementations

This pull request can be merged automatically.

You are not authorized to merge this pull request.

You can also view command line instructions.