check_explainer_count.py finds the ROADMAP table cell by column position

Closed Beginner friendly
#1,257 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
80/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
python
Domain
tooling

Research direction

Start in scripts/check_explainer_count.py, where the regex counts leading columns to find the Explainers cell in the ROADMAP Current traction table. Change it to read the header row, locate the Explainers column by name, and take that cell from the next data row. Done when a test that adds a column before Explainers still reads the correct cell.

Written by the indexing model from the issue text.

Description

enhancement tooling

Where: scripts/check_explainer_count.py.

Current gap: The regex for the Current traction table's Explainers cell counts the eight leading columns, so adding or reordering a column makes it match the wrong cell or nothing.

Suggested approach: Read the header row, find the Explainers column by name and take that cell of the next data row.

Done when: a test adds a column before Explainers and the check still reads the right cell.

Dominant language
HTML
Stars
47
Forks
45
Avg merge
3h 24m
Merged PRs (30d)
101

Getting set up

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.

More from yakew7/Fair-Code

All issues in yakew7/Fair-Code

Similar issues

More DevTools issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.