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

📝 Add docstrings to debug-log-level #3336

Closed

Conversation

coderabbitai[bot]
Copy link
Contributor

@coderabbitai coderabbitai bot commented Mar 12, 2025

Docstrings generation was requested by @VictoriousRaptor.

The following files were modified:

  • Flow.Launcher.Infrastructure/Logger/Log.cs
  • Flow.Launcher/App.xaml.cs
ℹ️ Note

CodeRabbit cannot perform edits on its own pull requests yet.

Docstrings generation was requested by @VictoriousRaptor.

* #2739 (comment)

The following files were modified:

* `Flow.Launcher.Infrastructure/Logger/Log.cs`
* `Flow.Launcher/App.xaml.cs`
@coderabbitai coderabbitai bot requested a review from VictoriousRaptor March 12, 2025 00:57
@prlabeler prlabeler bot added the bug Something isn't working label Mar 12, 2025
Copy link

gitstream-cm bot commented Mar 12, 2025

🥷 Code experts: Jack251970, VictoriousRaptor

Jack251970, VictoriousRaptor have most 👩‍💻 activity in the files.
Jack251970, theClueless have most 🧠 knowledge in the files.

See details

Flow.Launcher.Infrastructure/Logger/Log.cs

Activity based on git-commit:

Jack251970 VictoriousRaptor
MAR 2 additions & 0 deletions
FEB
JAN
DEC
NOV
OCT

Knowledge based on git-blame:
theClueless: 35%

Flow.Launcher/App.xaml.cs

Activity based on git-commit:

Jack251970 VictoriousRaptor
MAR
FEB 79 additions & 40 deletions
JAN 86 additions & 66 deletions
DEC
NOV
OCT

Knowledge based on git-blame:
Jack251970: 35%

To learn more about /:\ gitStream - Visit our Docs

Copy link

@check-spelling-bot Report

🔴 Please review

See the 📂 files view, the 📜action log, or 📝 job summary for details.

❌ Errors Count
❌ forbidden-pattern 22
⚠️ ignored-expect-variant 1
⚠️ non-alpha-in-dictionary 19

See ❌ Event descriptions for more information.

Using only_check_changed_files is incompatible with expect.txt.
To accept the items listed, you should add them to allow.txt.

If the flagged items are 🤯 false positives

If items relate to a ...

  • binary file (or some other file you wouldn't want to check at all).

    Please add a file path to the excludes.txt file matching the containing file.

    File paths are Perl 5 Regular Expressions - you can test yours before committing to verify it will match your files.

    ^ refers to the file's path from the root of the repository, so ^README\.md$ would exclude README.md (on whichever branch you're using).

  • well-formed pattern.

    If you can write a pattern that would match it,
    try adding it to the patterns.txt file.

    Patterns are Perl 5 Regular Expressions - you can test yours before committing to verify it will match your lines.

    Note that patterns can't match multiline strings.

Copy link

gitstream-cm bot commented Mar 12, 2025

Be a legend 🏆 by adding a before and after screenshot of the changes you made, especially if they are around UI/UX.

@VictoriousRaptor
Copy link
Contributor

well seems working but not very necessary

@VictoriousRaptor VictoriousRaptor deleted the coderabbitai/docstrings/8d3fd75 branch March 13, 2025 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant