test_all_files_already_beautiful contains wisdom beautify_changed_files_in_branch needs
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 48/100
Research direction
Start at line 36 of python_cc_reader/test_all_files_already_beautiful.py and compare the warning shown there with the behavior of beautify_changed_files_in_branch. Determine when the local beautification fails on a CxxTest file, then make the relevant failure provide the same useful warning. Verify that the warning appears for the intended failure and not unrelated cases.
Written by the indexing model from the issue text.
Description
print "One of the most likely reasons your file doesn't compile is that you have used a macro (e.g. TS_ASSERT) but have not followed it with a semicolon"
This warning is absolutely brilliant, and it is a shame that the user can only see it after the TEST SERVER fails to beautify. If they run beautify_changed_files_in_branch locally, it just tells them it can't do it because of mismatched {} without the helpful error message. Perhaps the message can migrate.
(I can do the idiot's version of "just always print the message" but I bet it is a 2 minute job for @aleaverfay to do the slicker "only print it if it failed on a cxxtest file").
- Dominant language
- Python
- Stars
- 9
- Forks
- 4
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from RosettaCommons/tools
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
RosettaCommons/tools#110 · 1 comment ·
-
Difficulty 3/5 1-2 days Newbie friendliness 42/100
RosettaCommons/tools#68 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 45/100
RosettaCommons/tools#57 ·
-
rosetta_submit.py makes a bad assumption about command line format with flags filesMay be free again @everyday847 claimed this 3726 days ago, and no pull request is open. Open
RosettaCommons/tools#48 · 1 assignee ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
RosettaCommons/tools#41 · 1 comment ·
All issues in RosettaCommons/tools
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
NousResearch/hermes-agent#140958 ·
Maintainers usually reply within 1 day
-
Bisect issue233186Openbisect
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
dtcxzyw/llvm-bisect-service#548 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
sallowayma-git/IELTS-practice#248 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 65/100
anthropics/claude-plugins-official#6763 · 10 comments · 10 reactions ·
Maintainers usually reply within 3 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Maintainers usually reply within 1 day