Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

This PR showcases the app's functionality #15

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

NikkelM
Copy link
Owner

@NikkelM NikkelM commented Feb 5, 2024

Below you can find an exemplary run result of the app on the changes of this PR.

The output shown in the comment can be customized in many ways through the configuration file.

Copy link

todo-pr-checker bot commented Feb 5, 2024

There are 6 unresolved action items in this Pull Request:

.github/workflows/release.yml (1 action item):

# TODO: Many different file types are supported, but if you are missing one, feel free to open an issue!

README.md (3 action items):

<!-- TODO: The app will only report on TODO items in code comments, each line is only counted once. -->
Do you keep forgetting to resolve that one `// TODO:...` or fix the last ` # Bug...` before merging your Pull Requests?
<!--
The capitalization of the todo, FixMe or BUG does not matter.
If action items are located close to each other, the embedded code link will contain a range of lines,
as is happening here. It also does not matter where in the comment the TODO is located.
-->

app.rb (2 action items):

Todo-PR-Checker/app.rb

Lines 26 to 29 in e946834

=begin Not a BUG: Don't worry about words that contain action items
lines like this one will be ignored - it's really BUGGING me when action items are misreported!
The line above is only shown in the app's comment because the lines above and below contain a TODO
=end

Did I do good? Let me know by helping maintain this app!

@NikkelM NikkelM marked this pull request as draft February 5, 2024 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant