metabase / metabase/metabase

Hanlde whitespace edge cases for `metabase.lib.native/extract-template-tags`

Open
#39,378 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

.Backend .Team/QueryingPlatform .Wanted: MLv2 Type:Tech Debt
Dominant language
Clojure
Stars
49.3k
Forks
6.8k
Avg merge
1d 13h
Merged PRs (30d)
653

Description

https://github.com/metabase/metabase/blob/b159e81/test/metabase/lib/native_test.cljc#L29

https://github.com/metabase/metabase/blob/b159e81/test/metabase/lib/native_test.cljc#L36

https://github.com/metabase/metabase/blob/b159e81/test/metabase/lib/native_test.cljc#L57

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the linked cases in test/metabase/lib/native_test.cljc at lines 29, 36, and 57, then inspect the metabase.lib.native/extract-template-tags entry point. Determine the expected behavior for whitespace edge cases from the surrounding tests, add coverage for the missing cases, and run the relevant native tests to confirm they pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
clojure
Domain
backend, testing
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.