MetPX / MetPX/Sundew

Accept only takes one argument, needs to detect error

Open
#7 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
6
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Lucie got bit by a copy/paste error where an accept clause in a sender had a space in it. It caused warnings not to get translated until we caught it.

accept clauses only take one argument. It would be good if sundew indicated (error message? fail to start up) when there is a clear syntax error in an accept, or a reject.

was: https://sourceforge.net/p/metpx/bugs/29/

Contributor guide

No contributing guide indexed for this repository

Research direction

No files or tests are named. Locate the parser or startup entry point handling accept and reject clauses, then reproduce the copy/paste case where an accept clause contains a space. Done means a clear syntax error is reported or startup fails for invalid accept or reject clauses, with valid clauses still accepted.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
networking
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.