GitHub Action: check that every solution file has a header comment
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
Research direction
Start by reviewing existing files under .github/workflows/ and the layout of solutions/ to see how changed files can be identified in pull requests. Add a small script to check the first lines of changed solution files for Problem:, Source:, and Note: using # or // comments, then have the workflow fail with a clear list of files missing headers. Done when the check runs on pull requests and reports missing headers; matching practice<n>.<ext> is a bonus.
Written by the indexing model from the issue text.
Description
Every solution is supposed to start with a comment containing Problem, Source, and Note lines. Nothing enforces that yet.
Time: about an hour
Skills: GitHub Actions basics, and a bit of Python, Bash, or Node
What to build
- A workflow in
.github/workflows/that runs on pull requests - A small script that scans changed files under
solutions/and checks the first lines forProblem:,Source:, andNote:(accept#and//comment styles) - Fail the check with a clear message listing the files that are missing a header
Bonus: also check the file name matches practice<n>.<ext>.
Who can take this
Anyone. Beginners, experienced developers, students, and people who do not code. Any language, any level. If you want to help, you are welcome here.
How to contribute
- Comment on this issue so nobody else starts the same thing
- Fork the repo
- Clone your fork and create a branch
- Make your change
- Commit, push, and open a Pull Request
Never opened a pull request before? Start with First Contributions or watch this walkthrough.
Stuck on anything? Ask in the comments. Basic questions are fine.
If this repo helps you or you like where it is going, a star is a nice way to say thanks and helps other people find it. Entirely optional, and you do not need it to contribute.
- Dominant language
- Python
- Stars
- 3
- Forks
- 2
- Avg merge
- 2h 33m
- Merged PRs (30d)
- 6
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 Huerte/algo-solutions-python
-
5-minutes algorithm all-levels any-language beginner-friendly codewars easy good first issue hackerrank hacktoberfest help wanted interview-prep leetcode open-source students
Difficulty 1/5 Under an hour Newbie friendliness 92/100
Huerte/algo-solutions-python#144 · 1 comment ·
-
30-minutes community documentation enhancement good first issue hacktoberfest help wanted
Difficulty 1/5 Under an hour Newbie friendliness 90/100
Huerte/algo-solutions-python#143 ·
-
5-minutes all-levels beginner-friendly codewars documentation good first issue hackerrank hacktoberfest interview-prep leetcode no-code-needed open-source students
Difficulty 2/5 Under an hour Newbie friendliness 68/100
Huerte/algo-solutions-python#140 ·
-
15-minutes 5-minutes algorithm any-language beginner-friendly c cpp csharp easy go good first issue hacktoberfest help wanted java javascript kotlin python rust swift typescript
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Huerte/algo-solutions-python#138 ·
-
30-minutes algorithm all-levels beginner-friendly codewars data-structures documentation dynamic-programming good first issue hackerrank hacktoberfest help wanted interview-prep leetcode no-code-needed open-source students
Difficulty 2/5 1-3 hours Newbie friendliness 83/100
Huerte/algo-solutions-python#134 ·
All issues in Huerte/algo-solutions-python
Similar issues
-
Context/namespace resources fail in-cluster: "Invalid kube-config file. No configuration found."Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
bug documentation
Difficulty 1/5 Under an hour Newbie friendliness 88/100
awslabs/visual-asset-management-system#880 ·
Maintainers usually reply within 1 day
-
netbox status: needs owner type: documentation
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
netbox-community/netbox#23533 ·
Maintainers usually reply within 1 day
-
app: scripts status: backlog type: documentation
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
netbox-community/netbox-scripts#68 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 91/100
marin-community/marin#10765 ·
Maintainers usually reply within 1 day