Skip to content

Commit

Permalink
[test/spec] Fix metadata
Browse files Browse the repository at this point in the history
Fix lint errors

Update j8 TODOs
  • Loading branch information
Andy C committed Dec 27, 2023
1 parent 3458645 commit fa3c2f5
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 16 deletions.
3 changes: 0 additions & 3 deletions builtin/io_ysh.py
Expand Up @@ -21,9 +21,6 @@
from frontend import typed_args
from mycpp import mylib
from mycpp.mylib import log
from ysh import cpython

import yajl

from typing import TYPE_CHECKING, cast, Dict
if TYPE_CHECKING:
Expand Down
3 changes: 0 additions & 3 deletions core/ui.py
Expand Up @@ -9,8 +9,6 @@
"""
from __future__ import print_function

import sys

from _devbuild.gen.id_kind_asdl import Id, Id_t, Id_str
from _devbuild.gen.syntax_asdl import (
Token,
Expand All @@ -32,7 +30,6 @@
from mycpp.mylib import print_stderr, tagswitch
from data_lang import j8
from data_lang import qsn
from ysh import cpython

from typing import List, Optional, Any, cast, TYPE_CHECKING
if TYPE_CHECKING:
Expand Down
16 changes: 8 additions & 8 deletions data_lang/j8.py
Expand Up @@ -4,9 +4,6 @@
TODO:
- Int vs. Float
- match.LooksLikeInteger() is a shortcut
- Errors
- Figure out location info for parse errors - turn a position into a line and
column?
Expand All @@ -19,22 +16,25 @@
- Distinguish pretty-printing vs. data transfer
- SHOW_CYCLES and SHOW_NON_DATA
- Make the whole thing translate to C++
- Translate the whole thing to C++
- use Bjoern DFA for UTF-8 validation in printing and parsing
- move more of LexerDecoder out of pyj8.py? I think it can translate
- Many more tests
- Run JSONTestSuite
- Remove ysh/cpython.py entirely
- QSN maybe_shell_encode() is used for bash features
- Remove most of QSN
- QSN maybe_shell_encode() is used for bash features
- Remove shell_compat which does \\x00 instead of \\0
- Many more tests
- Run JSONTestSuite
Other
- PrettyPrinter uses hnode.asdl?
- color
- line wrapping -- do this later
- this is open to contribution
- would like CONTRIBUTORS here
- Harmonize the API in data_lang/qsn.py
- use mylib.BufWriter output
Expand Down
1 change: 1 addition & 0 deletions spec/builtin-bracket.test.sh
@@ -1,4 +1,5 @@
## oils_failures_allowed: 1
## compare_shells: dash bash mksh

#### zero args: [ ]
[ ] || echo false
Expand Down
3 changes: 1 addition & 2 deletions test/spec.sh
Expand Up @@ -233,8 +233,7 @@ builtin-getopts() {
}

builtin-bracket() {
sh-spec spec/builtin-bracket.test.sh \
${REF_SHELLS[@]} $OSH_LIST "$@"
run-file builtin-bracket "$@"
}

builtin-trap() {
Expand Down

0 comments on commit fa3c2f5

Please sign in to comment.