unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: john muhl <jm@pub.pink>
To: emacs-devel@gnu.org
Cc: Yuan Fu <casouri@gmail.com>
Subject: Re: emacs-30 0fd259d166c 1/4: Fix elixir-ts-mode's range query
Date: Mon, 09 Sep 2024 18:00:40 -0500	[thread overview]
Message-ID: <87ikv3gox8.fsf@pub.pink> (raw)
In-Reply-To: <20240909035253.077E8C41F21@vcs2.savannah.gnu.org> (Yuan Fu's message of "Sun, 8 Sep 2024 23:52:52 -0400 (EDT)")

[-- Attachment #1: Type: text/plain, Size: 58 bytes --]

The elixir mode tests started failing after this commit.


[-- Attachment #2: elixir-ts-mode-tests.log --]
[-- Type: text/plain, Size: 2114 bytes --]

Running 1 tests (2024-09-09 05:57:39+0200, selector `(not (or (tag :expensive-test) (tag :unstable) (tag :nativecomp)))')
Test elixir-ts-mode-test-indentation backtrace:
  signal(ert-test-failed (("Mismatch in test \"Embedded HEEx\", file /
  ert-fail(("Mismatch in test \"Embedded HEEx\", file /home/build/emac
  ert-test--erts-test(((code lambda nil (elixir-ts-mode) (indent-regio
  ert-test-erts-file("/home/build/emacs-build/src-minimal/test/lisp/pr
  #f(compiled-function () #<bytecode 0x4b9a44325c9832>)()
  #f(compiled-function () #<bytecode -0xfc329de6ddc2e9>)()
  handler-bind-1(#f(compiled-function () #<bytecode -0xfc329de6ddc2e9>
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name elixir-ts-mode-test-indentation :docu
  ert-run-or-rerun-test(#s(ert--stats :selector (not (or (tag :expensi
  ert-run-tests((not (or (tag :expensive-test) (tag :unstable) (tag :n
  ert-run-tests-batch((not (or (tag :expensive-test) (tag :unstable) (
  ert-run-tests-batch-and-exit((not (or (tag :expensive-test) (tag :un
  eval((ert-run-tests-batch-and-exit '(not (or (tag :expensive-test) (
  command-line-1(("-L" ":." "-l" "ert" "--eval" "(setq treesit-extra-l
  command-line()
  normal-top-level()
Test elixir-ts-mode-test-indentation condition:
    (ert-test-failed
     ("Mismatch in test \"Embedded HEEx\", file /home/build/emacs-build/src-minimal/test/lisp/progmodes/elixir-ts-mode-resources/indent.erts"
      #("defmodule Foo do\n  def foo(assigns) do\n    ~H\"\"\"\n<span>\n  text\n</span>\n    \"\"\"\n  end\nend\n"
	0 47 (fontified nil) 47 48 (syntax-table (8) fontified nil) 48 77
	(fontified nil) 77 78 (syntax-table (8) fontified nil) 78 89
	(fontified nil))
      "defmodule Foo do\n  def foo(assigns) do\n    ~H\"\"\"\n    <span>\n      text\n    </span>\n    \"\"\"\n  end\nend\n"))
   FAILED  1/1  elixir-ts-mode-test-indentation (0.066101 sec) at lisp/progmodes/elixir-ts-mode-tests.el:26

Ran 1 tests, 0 results as expected, 1 unexpected (2024-09-09 05:57:39+0200, 0.130395 sec)

1 unexpected results:
   FAILED  elixir-ts-mode-test-indentation


[-- Attachment #3: Type: text/plain, Size: 1087 bytes --]



Yuan Fu <casouri@gmail.com> writes:

> branch: emacs-30
> commit 0fd259d166c0dca4d097ad026ac748748ba88497
> Author: Yuan Fu <casouri@gmail.com>
> Commit: Yuan Fu <casouri@gmail.com>
>
>     Fix elixir-ts-mode's range query
>     
>     * lisp/progmodes/elixir-ts-mode.el:
>     (elixir-ts--treesit-range-rules): Add underscore in front of the name
>     capture, so Emacs won't put heex parser on it.
> ---
>  lisp/progmodes/elixir-ts-mode.el | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/lisp/progmodes/elixir-ts-mode.el b/lisp/progmodes/elixir-ts-mode.el
> index 9804152d9ab..815827ed13c 100644
> --- a/lisp/progmodes/elixir-ts-mode.el
> +++ b/lisp/progmodes/elixir-ts-mode.el
> @@ -572,7 +572,9 @@
>      (treesit-range-rules
>       :embed 'heex
>       :host 'elixir
> -     '((sigil (sigil_name) @name (:match "^[HF]$" @name) (quoted_content) @heex)))))
> +     '((sigil (sigil_name) @_name
> +              (:match "^[HF]$" @_name)
> +              (quoted_content) @heex)))))
>  
>  (defvar heex-ts--sexp-regexp)
>  (defvar heex-ts--indent-rules)

  parent reply	other threads:[~2024-09-09 23:00 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <172585397252.25544.14799687468267584593@vcs2.savannah.gnu.org>
     [not found] ` <20240909035253.23A45C41F2E@vcs2.savannah.gnu.org>
2024-09-09  3:58   ` emacs-30 3fcec09f754 2/4: Add debugging function for treesit.c Po Lu via Emacs development discussions.
2024-09-09  4:13     ` Yuan Fu
2024-09-09 11:55     ` Eli Zaretskii
     [not found] ` <20240909035253.077E8C41F21@vcs2.savannah.gnu.org>
2024-09-09 23:00   ` john muhl [this message]
2024-09-11  3:08     ` emacs-30 0fd259d166c 1/4: Fix elixir-ts-mode's range query Yuan Fu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87ikv3gox8.fsf@pub.pink \
    --to=jm@pub.pink \
    --cc=casouri@gmail.com \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).