Skip to content

Conversation

@mcoetzee
Copy link
Collaborator

@mcoetzee mcoetzee commented Sep 9, 2025

Follow up of #221

What does this change?

  • removes typescript (and tsconfig)
  • converts oev-auctioneer.test.ts-> oev-auctioneer.test.js
  • adds nicer error message when oev-auctioneer.md file cannot be found in the test
  • sets Prettier's printWidth to 120 for JS files

@mcoetzee mcoetzee requested a review from Siegrift September 9, 2025 10:43
@mcoetzee mcoetzee self-assigned this Sep 9, 2025
@github-actions
Copy link

github-actions bot commented Sep 9, 2025

Visit the preview URL for this PR (updated for commit 353833b):

https://oev-docs--pr227-remove-typescript-whz4avs6.web.app

(expires Tue, 16 Sep 2025 10:44:37 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6915b094b5ba83fde754632ba50c1ee9406d433f

Copy link
Collaborator

@Siegrift Siegrift left a comment

Choose a reason for hiding this comment

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

👍 LGTM

@mcoetzee mcoetzee merged commit 4a3c6a2 into main Sep 9, 2025
3 checks passed
@mcoetzee mcoetzee deleted the remove-typescript branch September 9, 2025 15:18
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.

3 participants