Skip to content
This repository has been archived by the owner on Mar 15, 2021. It is now read-only.

feat: display specberus errors in more friendly way #17

Merged
merged 1 commit into from
Jun 6, 2020

Conversation

sidvishnoi
Copy link
Member

@sidvishnoi sidvishnoi commented Jun 5, 2020

Closes #16
image
image
image

@sidvishnoi sidvishnoi marked this pull request as draft June 5, 2020 10:53
@sidvishnoi sidvishnoi marked this pull request as ready for review June 5, 2020 11:48
@@ -0,0 +1,239 @@
exports.messages = {
Copy link
Member Author

Choose a reason for hiding this comment

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

This file is cloned from w3c/specberus repo using specberus.sh script.

@@ -0,0 +1,1943 @@
{
Copy link
Member Author

Choose a reason for hiding this comment

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

This file is also cloned from w3c/specberus repo using specberus.sh script.

@@ -0,0 +1,81 @@
// @ts-check
// Based on https://github.com/w3c/specberus/blob/master/lib/l10n.js
Copy link
Member Author

Choose a reason for hiding this comment

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

Tries to make messages cleaner for CI env (less HTML) and simplifies some code.

Copy link
Member

@marcoscaceres marcoscaceres left a comment

Choose a reason for hiding this comment

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

Forgot you would have to endure the l10 lib from specberus 😹

@sidvishnoi sidvishnoi merged commit 8e21a00 into master Jun 6, 2020
@sidvishnoi sidvishnoi deleted the specberus-errors branch June 6, 2020 14:15
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.

Display echidna errors
2 participants