Skip to content

Commit

Permalink
Remove concept of prev attach markers as it just doesn't seem to be n…
Browse files Browse the repository at this point in the history
…eeded
  • Loading branch information
paulfioravanti committed Sep 25, 2023
1 parent 35554db commit 3864a8d
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 7 deletions.
4 changes: 0 additions & 4 deletions plover_q_and_a/config/load.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,6 @@ def load(config_filepath: Path) -> dict[str, any]:
"BYLINE_FOLLOWING_STATEMENT_FOR": lambda speaker_name: (
statement_end + formatted_byline(speaker_name)
),
"PREV_ATTACH_MARKERS": [
_question_end_marker(data),
_statement_end_marker(data)
],
"QUESTION": formatted_question,
"QUESTION_FOLLOWING_INTERRUPT": interrupt + formatted_question,
"QUESTION_FOLLOWING_QUESTION": question_end + formatted_question,
Expand Down
3 changes: 1 addition & 2 deletions plover_q_and_a/extension.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,8 +86,7 @@ def _q_and_a(self, ctx: _Context, args: str) -> _Action:
else:
text = sign.text(args, self._config)
action.text = text
if text[0] in self._config["PREV_ATTACH_MARKERS"]:
action.prev_attach = True
action.prev_attach = True
action.next_attach = True
action.next_case = Case.CAP_FIRST_WORD

Expand Down
1 change: 0 additions & 1 deletion test/config/test_load.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
"ANSWER_FOLLOWING_INTERRUPT",
"ANSWER_FOLLOWING_QUESTION",
"ANSWER_FOLLOWING_STATEMENT",
"PREV_ATTACH_MARKERS",
"QUESTION",
"QUESTION_FOLLOWING_INTERRUPT",
"QUESTION_FOLLOWING_QUESTION",
Expand Down

0 comments on commit 3864a8d

Please sign in to comment.