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.

Sign up for GitHub

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

Jump to bottom

restore default loglevel to INFO #255

Merged
merged 6 commits into from
Oct 14, 2023
Merged

restore default loglevel to INFO #255

merged 6 commits into from
Oct 14, 2023

Conversation

Mic92
Copy link
Member

@Mic92 Mic92 commented Oct 14, 2023

No description provided.

@Mic92
Copy link
Member Author

Mic92 commented Oct 14, 2023

Before:

╭─[~/.homesick/repos/dotfiles/nix-merge-bot]─[main]
╰─ ╰─ % nix fmt
[ERROR] #python's formatter failed: exit status 1

After:

╭─[~/.homesick/repos/dotfiles/nix-merge-bot]─[main]
╰─ ╰─ % /home/joerg/git/treefmt/target/debug/treefmt --config-file /nix/store/d2d9kkjk17j8474wffxlfg4dczyg5hhb-treefmt.toml
[WARN ] cache: failed to load the manifest due to: No such file or directory (os error 2)
[INFO ] #nixpkgs-fmt: 2 files processed in 2.61ms
[INFO ] #deno: 1 files processed in 7.27ms
[WARN ] Error using formatter #python:
    • [STDOUT]:
    nix_merge_bot/webhook/handler.py:40:9: N802 Function name `do_GET` should be lowercase
    nix_merge_bot/webhook/handler.py:74:9: N802 Function name `do_POST` should be lowercase
    Found 2 errors.

    • [STDERR]:
    + /nix/store/47yzh9v6m6h46qg4ki8qidaf6qaknndf-ruff-0.0.292/bin/ruff --fix /home/joerg/.homesick/repos/dotfiles/nix-merge-bot/nix_merge_bot/__init__.py /home/joerg/.homesick/repos/dotfiles/nix-merge-bot/nix_merge_bot/github.py /home/joerg/.homesick/repos/dotfiles/nix-merge-bot/nix_merge_bot/nix.py /home/joerg/.homesick/repos/dotfiles/nix-merge-bot/nix_merge_bot/server.py /home/joerg/.homesick/repos/dotfiles/nix-merge-bot/nix_merge_bot/webhook/__init__.py /home/joerg/.homesick/repos/dotfiles/nix-merge-bot/nix_merge_bot/webhook/errors.py /home/joerg/.homesick/repos/dotfiles/nix-merge-bot/nix_merge_bot/webhook/handler.py /home/joerg/.homesick/repos/dotfiles/nix-merge-bot/nix_merge_bot/webhook/httpheader.py /home/joerg/.homesick/repos/dotfiles/nix-merge-bot/nix_merge_bot/webhook/secret.py

[WARN ] Error using formatter #mypy-.:
    • [STDOUT]:
    nix_merge_bot/webhook/secret.py:9: error: Function is missing a return type annotation  [no-untyped-def]
    nix_merge_bot/server.py:5: error: Module "nix_merge_bot.webhook" has no attribute "GithubWebHook"  [attr-defined]
    nix_merge_bot/webhook/handler.py:46: error: Signature of "send_error" incompatible with supertype "BaseHTTPRequestHandler"  [override]
    nix_merge_bot/webhook/handler.py:46: note:      Superclass:
    nix_merge_bot/webhook/handler.py:46: note:          def send_error(self, code: int, message: str | None = ..., explain: str | None = ...) -> None
    nix_merge_bot/webhook/handler.py:46: note:      Subclass:
    nix_merge_bot/webhook/handler.py:46: note:          def send_error(self, code: int, message: str) -> None
    nix_merge_bot/webhook/handler.py:88: error: Argument 1 to "validate_signature" of "WebhookSecret" has incompatible type "bytes"; expected "str"  [arg-type]
    nix_merge_bot/webhook/handler.py:88: error: Argument 2 to "validate_signature" of "WebhookSecret" has incompatible type "Message"; expected "dict[str, str]"  [arg-type]
    nix_merge_bot/nix.py:31: error: Function is missing a return type annotation  [no-untyped-def]
    Found 6 errors in 4 files (checked 9 source files)

    • [STDERR]:
    + shopt -s globstar
    + cd .
    + export PYTHONPATH=/nix/store/pzf6dnxg8gf04xazzjdwarm7s03cbrgz-python3-3.10.12/lib/python3.10/site-packages
    + PYTHONPATH=/nix/store/pzf6dnxg8gf04xazzjdwarm7s03cbrgz-python3-3.10.12/lib/python3.10/site-packages

@Mic92
Copy link
Member Author

Mic92 commented Oct 14, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Oct 14, 2023

queue

🛑 The pull request has been removed from the queue default

Unexpected queue change: the updated pull request #255 has been manually updated.

You can take a look at Queue: Embarked in merge queue check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@Mic92
Copy link
Member Author

Mic92 commented Oct 14, 2023

@mergify requeue

@mergify
Copy link
Contributor

mergify bot commented Oct 14, 2023

requeue

❌ This pull request head commit has not been previously disembarked from queue.

@Mic92
Copy link
Member Author

Mic92 commented Oct 14, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Oct 14, 2023 •
edited
Loading

queue

🛑 The pull request has been removed from the queue default

The queue conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge queue check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@Mic92
Copy link
Member Author

Mic92 commented Oct 14, 2023

@mergify rebase

@mergify
Copy link
Contributor

mergify bot commented Oct 14, 2023

rebase

✅ Branch has been successfully rebased

@Mic92
flake.lock: Update
dee9815 
Flake lock file updates:

• Updated input 'flake-parts':
    'github:hercules-ci/flake-parts/8d0e2444ab05f79df93b70e5e497f8c708eb6b9b' (2022-12-07)
  → 'github:hercules-ci/flake-parts/c9afaba3dfa4085dbd2ccb38dfade5141e33d9d4' (2023-10-03)
• Updated input 'mkdocs-numtide':
    'github:numtide/mkdocs-numtide/af6c4a5f7c0a59da3b557795f57dcae5707523ac' (2023-04-01)
  → 'github:numtide/mkdocs-numtide/b3008171c75083f2bf2f1dc4e6781d4737dfaa49' (2023-06-26)
• Updated input 'nixpkgs':
    'github:NixOS/nixpkgs/4172cdda7e56a48065475fb98c57b03b83c1fde4' (2022-12-16)
  → 'github:NixOS/nixpkgs/01441e14af5e29c9d27ace398e6dd0b293e25a54' (2023-10-11)
• Updated input 'rust-overlay':
    'github:oxalica/rust-overlay/7da2f6b3a0c32f661cb2864d7fbd1d7e6f0c7543' (2022-12-16)
  → 'github:oxalica/rust-overlay/dce60ca7fca201014868c08a612edb73a998310f' (2023-10-14)
• Updated input 'rust-overlay/flake-utils':
    'github:numtide/flake-utils/c0e246b9b83f637f4681389ecabcb2681b4f3af0' (2022-08-07)
  → 'github:numtide/flake-utils/cfacdce06f30d2b68473a46042957675eebb3401' (2023-04-11)
• Added input 'rust-overlay/flake-utils/systems':
    'github:nix-systems/default/da67096a3b9bf56a91d16901293e51ba5b49a27e' (2023-04-09)
• Updated input 'systems':
    'github:nix-systems/default/4e9a51a15ceb27e5141819142a7d2ee827943fc8' (2023-04-08)
  → 'github:nix-systems/default/da67096a3b9bf56a91d16901293e51ba5b49a27e' (2023-04-09)
@Mic92
upgrade to new flake-parts syntax
4b01279
@Mic92
work around terraform beeing marked as unfree
fbf17c5
@Mic92
fix formatting of index.html
ee6bc6b
@Mic92
Copy link
Member Author

Mic92 commented Oct 14, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Oct 14, 2023 •
edited
Loading

queue

🛑 Branch protection settings are not validated anymore

Branch protection is enabled and is preventing Mergify to merge the pull request. Mergify will merge when branch protection settings validate the pull request once again. (detail: You're not authorized to push to this branch. Visit https://docs.github.com/articles/about-protected-branches/ for more information.)

@Mic92
Copy link
Member Author

Mic92 commented Oct 14, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Oct 14, 2023

queue

🛑 The pull request has been removed from the queue default

Pull request #255 has been dequeued due to failing checks or checks timeout.

You can take a look at Queue: Embarked in merge queue check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@Mic92
Copy link
Member Author

Mic92 commented Oct 14, 2023

@mergify requeue

@mergify
Copy link
Contributor

mergify bot commented Oct 14, 2023

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically

@Mic92
Copy link
Member Author

Mic92 commented Oct 14, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Oct 14, 2023 •
edited
Loading

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 90e98e0

@mergify mergify bot merged commit 90e98e0 into main Oct 14, 2023
17 checks passed
@mergify mergify bot deleted the log-level-check branch October 14, 2023 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aldoborrero aldoborrero Awaiting requested review from aldoborrero

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Mic92