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

format not solvable error messages #1886

Merged
merged 6 commits into from
Oct 10, 2023
Merged

format not solvable error messages #1886

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
4a8b202
format not solvable error messages
jaimergp Oct 10, 2023
42c6623
one block per platform
jaimergp Oct 10, 2023
795247f
missing closing tag
jaimergp Oct 10, 2023
b9b5c69
align left for prettier lines
jaimergp Oct 10, 2023
d57fcfe
do not use summary (toggle button missing in this style)
jaimergp Oct 10, 2023
cfb2452
do not generate URL if none is passed; pre-commit
jaimergp Oct 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 15 additions & 5 deletions conda_forge_tick/auto_tick.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
import typing
import tqdm
from subprocess import CalledProcessError
from textwrap import dedent

Check warning on line 16 in conda_forge_tick/auto_tick.py

View check run for this annotation

Codecov / codecov/patch

conda_forge_tick/auto_tick.py#L16 

Added line #L16 was not covered by tests
from typing import (
Optional,
MutableSequence,
Expand Down Expand Up @@ -363,11 +364,20 @@
verbosity=2,
)
if not solvable:
_solver_err_str = "not solvable ({}): {}: {}".format(
('<a href="' + os.getenv("CIRCLE_BUILD_URL", "") + '">bot CI job</a>'),
base_branch,
sorted(set(errors)),
)
ci_url = os.getenv("CIRCLE_BUILD_URL")
ci_url = f"(<a href='{ci_url}'>bot CI job</a>)" if ci_url else ""
_solver_err_str = dedent(

Check warning on line 369 in conda_forge_tick/auto_tick.py

View check run for this annotation

Codecov / codecov/patch

conda_forge_tick/auto_tick.py#L367-L369 

Added lines #L367 - L369 were not covered by tests
f"""
not solvable {ci_url} @ {base_branch}
<details>
jaimergp marked this conversation as resolved.
Show resolved Hide resolved
<div align="left">
<pre>
{'</pre><pre>'.join(sorted(set(errors)))}
</pre>
</div>
</details>
""",
).strip()

if isinstance(migrator, Version):
with feedstock_ctx.attrs["version_pr_info"] as vpri:
Expand Down
Loading