Brent Westbrook
40fd52dde0
Exclude broken symlinks from meson-python ecosystem check ( #17993 )
...
CI / Fuzz for new ty panics (push) Blocked by required conditions
CI / cargo shear (push) Blocked by required conditions
CI / Determine changes (push) Waiting to run
CI / cargo fmt (push) Waiting to run
CI / cargo clippy (push) Blocked by required conditions
CI / cargo test (linux) (push) Blocked by required conditions
CI / cargo test (linux, release) (push) Blocked by required conditions
CI / cargo test (windows) (push) Blocked by required conditions
CI / cargo test (wasm) (push) Blocked by required conditions
CI / cargo build (release) (push) Waiting to run
CI / cargo build (msrv) (push) Blocked by required conditions
CI / cargo fuzz build (push) Blocked by required conditions
CI / fuzz parser (push) Blocked by required conditions
CI / test scripts (push) Blocked by required conditions
CI / ecosystem (push) Blocked by required conditions
CI / python package (push) Waiting to run
CI / pre-commit (push) Waiting to run
CI / mkdocs (push) Waiting to run
CI / formatter instabilities and black similarity (push) Blocked by required conditions
CI / test ruff-lsp (push) Blocked by required conditions
CI / check playground (push) Blocked by required conditions
CI / benchmarks (push) Blocked by required conditions
[ty Playground] Release / publish (push) Waiting to run
Summary
--
This should resolve the formatter ecosystem errors we've been seeing
lately. https://github.com/mesonbuild/meson-python/pull/728 added the
links, which I think are intentionally broken for testing purposes.
Test Plan
--
Ecosystem check on this PR
2025-05-09 16:59:00 -04:00
Brent Westbrook
67602512b6
Recognize SyntaxError: as an error code for ecosystem checks ( #16879 )
...
Summary
--
This updates the regex in `ruff-ecosystem` to catch syntax errors in an
effort to prevent bugs like #16874 . This should catch `ParseError`s,
`UnsupportedSyntaxError`s, and the upcoming `SemanticSyntaxError`s.
Test Plan
--
I ran the ecosystem check locally comparing v0.11.0 and v0.11.1 and saw
a large number (2757!) of new syntax errors. I also manually tested the
regex on a few lines before that.
If we merge this before #16878 , I'd expect to see that number decrease
substantially in that PR too, as another test.
2025-03-20 17:25:40 -04:00
Dhruv Manilawala
613951d848
Ignore notebook with syntax error in ecosystem ( #15604 )
...
I'll open a PR in `openai/openai-cookbook` to fix these syntax errors
later but for now let's just ignore the notebook.
<img width="2560" alt="Screenshot 2025-01-20 at 10 17 43 AM"
src="https://github.com/user-attachments/assets/075127e4-956d-43f9-b175-b492600dbdee "
/>
2025-01-20 04:53:22 +00:00
renovate[bot]
070c160fd5
Update dependency tomli_w to v1.2.0 ( #15600 )
...
This PR contains the following updates:
| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [tomli_w](https://redirect.github.com/hukkin/tomli-w )
([changelog](https://redirect.github.com/hukkin/tomli-w/blob/master/CHANGELOG.md ))
| `==1.1.0` -> `==1.2.0` |
[](https://docs.renovatebot.com/merge-confidence/ )
|
[](https://docs.renovatebot.com/merge-confidence/ )
|
[](https://docs.renovatebot.com/merge-confidence/ )
|
[](https://docs.renovatebot.com/merge-confidence/ )
|
---
> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.
---
### Release Notes
<details>
<summary>hukkin/tomli-w (tomli_w)</summary>
###
[`v1.2.0`](https://redirect.github.com/hukkin/tomli-w/blob/HEAD/CHANGELOG.md#120 )
[Compare
Source](https://redirect.github.com/hukkin/tomli-w/compare/1.1.0...1.2.0 )
- Added
- Always preserve `decimal.Decimal` type in parse round trips
- Improved
- Better `TypeError` message on invalid mapping keys
- Unify type representation in error messages
- Performance
- Improve import time by removing `typing` import
- Improve import time by removing `string` import
- Improve import time by lazy importing `decimal`
</details>
---
### Configuration
📅 **Schedule**: Branch creation - "before 4am on Monday" (UTC),
Automerge - At any time (no schedule defined).
🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.
♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.
🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.
---
- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box
---
This PR was generated by [Mend Renovate](https://mend.io/renovate/ ).
View the [repository job
log](https://developer.mend.io/github/astral-sh/ruff ).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4xMDcuMCIsInVwZGF0ZWRJblZlciI6IjM5LjEwNy4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJpbnRlcm5hbCJdfQ==-->
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2025-01-20 10:18:43 +05:30
Charlie Marsh
5567e7c26b
Use uv consistently throughout the documentation ( #15302 )
...
## Summary
Closes
https://github.com/astral-sh/ruff/issues/15301#issuecomment-2573350821 .
2025-01-07 14:43:25 +00:00
Enric Calabuig
bafe8714a8
Don't fix in ecosystem check ( #15267 )
CI / Determine changes (push) Waiting to run
CI / cargo fmt (push) Waiting to run
CI / cargo clippy (push) Blocked by required conditions
CI / cargo test (linux) (push) Blocked by required conditions
CI / cargo test (linux, release) (push) Blocked by required conditions
CI / cargo test (windows) (push) Blocked by required conditions
CI / cargo test (wasm) (push) Blocked by required conditions
CI / cargo build (release) (push) Waiting to run
CI / cargo build (msrv) (push) Blocked by required conditions
CI / cargo fuzz build (push) Blocked by required conditions
CI / fuzz parser (push) Blocked by required conditions
CI / test scripts (push) Blocked by required conditions
CI / ecosystem (push) Blocked by required conditions
CI / cargo shear (push) Blocked by required conditions
CI / python package (push) Waiting to run
CI / pre-commit (push) Waiting to run
CI / mkdocs (push) Waiting to run
CI / formatter instabilities and black similarity (push) Blocked by required conditions
CI / test ruff-lsp (push) Blocked by required conditions
CI / benchmarks (push) Blocked by required conditions
2025-01-06 10:21:34 +01:00
renovate[bot]
48d33595b9
Update dependency tomli to v2.2.1 ( #14718 )
2024-12-02 01:22:18 +00:00
InSync
e5c7d87461
Add @astropy/astropy to ecosystem checks ( #14565 )
2024-11-24 12:47:11 +01:00
Micha Reiser
e9079e7d95
Remove the deprecated E999 rule code ( #14428 )
2024-11-20 13:11:51 +01:00
renovate[bot]
80f5cdcf66
Update dependency tomli to v2.1.0 ( #14418 )
2024-11-18 01:56:05 +00:00
renovate[bot]
28e995023d
Update dependency tomli_w to v1.1.0 ( #13849 )
...
This PR contains the following updates:
| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [tomli_w](https://redirect.github.com/hukkin/tomli-w )
([changelog](https://redirect.github.com/hukkin/tomli-w/blob/master/CHANGELOG.md ))
| `==1.0.0` -> `==1.1.0` |
[](https://docs.renovatebot.com/merge-confidence/ )
|
[](https://docs.renovatebot.com/merge-confidence/ )
|
[](https://docs.renovatebot.com/merge-confidence/ )
|
[](https://docs.renovatebot.com/merge-confidence/ )
|
---
### Release Notes
<details>
<summary>hukkin/tomli-w (tomli_w)</summary>
###
[`v1.1.0`](https://redirect.github.com/hukkin/tomli-w/blob/HEAD/CHANGELOG.md#110 )
[Compare
Source](https://redirect.github.com/hukkin/tomli-w/compare/1.0.0...1.1.0 )
- Removed
- Support for Python 3.7 and 3.8
- Added
- Accept generic `collections.abc.Mapping`, not just `dict`, as input.
Thank you [Watal M. Iwasaki](https://redirect.github.com/heavywatal ) for
the
[PR](https://redirect.github.com/hukkin/tomli-w/pull/46 ).
- `indent` keyword argument for customizing indent width of arrays.
Thank you [Wim Jeantine-Glenn](https://redirect.github.com/wimglenn ) for
the
[PR](https://redirect.github.com/hukkin/tomli-w/pull/49 ).
- Type annotations
- Type annotate `dump` function's output stream object as
`typing.IO[bytes]` (previously `typing.BinaryIO`)
</details>
---
### Configuration
📅 **Schedule**: Branch creation - "before 4am on Monday" (UTC),
Automerge - At any time (no schedule defined).
🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.
♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.
🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.
---
- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box
---
This PR was generated by [Mend Renovate](https://mend.io/renovate/ ).
View the [repository job
log](https://developer.mend.io/github/astral-sh/ruff ).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4xMjAuMSIsInVwZGF0ZWRJblZlciI6IjM4LjEyMC4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJpbnRlcm5hbCJdfQ==-->
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2024-10-21 09:05:30 +05:30
renovate[bot]
98878c9bf2
Update dependency tomli to v2.0.2 ( #13649 )
...
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2024-10-07 10:16:43 +01:00
renovate[bot]
346dbf45b5
Update pre-commit dependencies ( #13289 )
...
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Alex Waygood <alex.waygood@gmail.com>
2024-09-09 11:11:01 +00:00
Dhruv Manilawala
f4bed22b05
Ignore invalid notebook in ecosystem checks ( #13211 )
2024-09-02 16:22:51 +05:30
Dhruv Manilawala
89c8b49027
Update OpenAI excluded notebooks from ecosystem checks ( #12867 )
...
## Summary
Follow-up to #12864 , we don't need to exclude these notebooks anymore.
## Test plan
- [x] Make sure that ecosystem checks are green.
2024-08-14 08:03:25 +05:30
ember91
9296bd4e3f
Fix a typo ( #12633 )
...
Co-authored-by: Emil Berg <emil.berg@ericsson.com>
2024-08-02 16:39:27 +01:00
Chris Krycho
c1bc7f4dee
Remove ecosystem_ci flag from Ruff CLI ( #12596 )
...
## Summary
@zanieb noticed while we were discussing #12595 that this flag is now
unnecessary, so remove it and the flags which reference it.
## Test Plan
Question for maintainers: is there a test to add *or* remove here? (I’ve
opened this as a draft PR with that in view!)
2024-07-31 11:40:03 -05:00
Micha Reiser
3ace12943e
Ignore more open ai notebooks for now ( #12448 )
2024-07-22 14:16:48 +02:00
Micha Reiser
b1487b6b4f
Ignore more OpenAI notebooks with syntax errors in the ecosystem check ( #12342 )
2024-07-16 10:32:00 +02:00
Alex Waygood
d9c15e7a12
[Ecosystem checks] trio has changed its default branch name to main ( #12225 )
...
Fixes CI errors seen in
https://github.com/astral-sh/ruff/pull/12224#issuecomment-2212594024
x-ref
17b3644f64
2024-07-07 23:37:27 +01:00
Dhruv Manilawala
ee90017d3f
Remove demisto/content from ecosystem checks ( #12160 )
...
## Summary
Follow-up to https://github.com/astral-sh/ruff/pull/12129 to remove the
`demisto/content` from ecosystem checks. The previous PR removed it from
the deprecated script which I didn't notice until recently.
## Test Plan
Ecosystem comment
2024-07-03 08:25:29 +00:00
Alex Waygood
0a24d70bfd
[Ruff v0.5] Fix ZeroDivisionErrors in the ecosystem check ( #12027 )
...
Seen in CI in https://github.com/astral-sh/ruff/pull/12026
2024-06-27 13:44:11 +02:00
Akshet Pandey
4a305588e9
[flake8-bandit] request-without-timeout should warn for requests.request ( #11548 )
...
## Summary
Update
[S113](https://docs.astral.sh/ruff/rules/request-without-timeout/ ) to
also warns for missing timeout on when calling `requests.request`
2024-05-28 16:31:12 +00:00
Zanie Blue
46fcd19ca6
Fix division by zero error in ecosystem check ( #11469 )
...
e.g.
25143076890df7cbcd
2024-05-19 09:08:10 -05:00
Charlie Marsh
d9ec3d56b0
Add some new projects to the ecosystem CI ( #11468 )
...
Co-authored-by: Zanie Blue <contact@zanie.dev>
2024-05-19 08:08:38 -05:00
Charlie Marsh
dd6d411026
Remove comma from ecosystem checks ( #11466 )
...
## Summary
Something's up with this repo -- they added a post-checkout hook? So
let's just remove it for now. We should go through and add a new batch
of repositories some time.
2024-05-18 23:37:56 -04:00
renovate[bot]
388658efdb
Update pre-commit dependencies ( #10698 )
...
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Zanie Blue <contact@zanie.dev>
Co-authored-by: Alex Waygood <alex.waygood@gmail.com>
2024-04-06 23:00:41 +00:00
Alex Waygood
59ac3f48c8
Select more rules for typeshed in the ecosystem checks ( #10564 )
2024-03-25 14:04:11 +00:00
Zanie Blue
0a99bd84ce
Add PlasmaPy to ecosystem checks ( #10530 )
...
ref
https://github.com/astral-sh/ruff/issues/10528#issuecomment-2015805710
2024-03-22 15:28:13 -05:00
Zanie Blue
fb05d218c3
Skip another invalid notebook in the OpenAI repository ( #10209 )
...
We should consider another source for notebook ecosystem checks, these
constantly have syntax errors
2024-03-03 09:41:31 -06:00
Micha Reiser
0373b51823
Remove indico ecosystem override ( #10180 )
2024-03-01 10:38:13 +01:00
Micha Reiser
72ccb34ba6
Fix ecosystem check for indico ( #10164 )
2024-02-29 10:27:33 +01:00
Charlie Marsh
75553ab1c0
Remove ecosystem failures ( #9854 )
...
## Summary
These are kinda disruptive, I'd prefer to TODO unless someone is
interested in solving them ASAP.
2024-02-06 09:45:13 -05:00
Adrian
83195a6030
ruff-ecosystem: Add indico/indico repo ( #9850 )
...
It's a pretty big codebase using lots of different stuff, so a good
candidate for finding obscure problems.
I didn't look more closely which options are used (I have the feeling
`--select ALL` is not implied, since I see you adding it via
`check_options` for certain entries but not for others), the repo itself
has a pretty large ruff.toml - but assuming ecosystem just cares about
differences between base and head of a PR, `ALL` most likely makes
sense.
2024-02-06 00:37:58 +00:00
Zanie
e37b3b0742
Always request the concise output format during ecosystem checks ( #9708 )
...
Fixes a regression in the ecosystem checks from
https://github.com/astral-sh/ruff/pull/9687 which was causing them to
run for multiple hours due to the size of the output.
We need the concise format for comparisons.
We should probably update the ecosystem checks to actually diff the full
output in the future because that'd be nice.
# Conflicts:
# python/ruff-ecosystem/ruff_ecosystem/projects.py
2024-02-01 13:35:02 -06:00
Zanie Blue
f18e7d40ac
Add internal hidden rules for testing ( #9747 )
...
Updated implementation of https://github.com/astral-sh/ruff/pull/7369
which was left out in the cold.
This was motivated again following changes in #9691 and #9689 where we
could not test the changes without actually deprecating or removing
rules.
---
Follow-up to discussion in https://github.com/astral-sh/ruff/pull/7210
Moves integration tests from using rules that are transitively in
nursery / preview groups to dedicated test rules that only exist during
development. These rules always raise violations (they do not require
specific file behavior). The rules are not available in production or in
the documentation.
Uses features instead of `cfg(test)` for cross-crate support per
https://github.com/rust-lang/cargo/issues/8379
2024-02-01 08:44:51 -06:00
Zanie Blue
a42600e9a2
Always unset the required-version option during ecosystem checks ( #9593 )
...
Uses our existing configuration overrides to unset the
`required-version` option in ecosystem projects during checks.
The downside to this approach, is we will now update the config file of
_every_ project (with a config file). This roughly normalizes the configuration file, as we
don't preserve comments and such. We could instead do a more targeted
approach applying this override to projects which we know use this
setting 🤷♀️
2024-01-20 23:07:11 -06:00
Zanie Blue
4b8b3a1ced
Add jupyter notebooks to ecosystem checks ( #9293 )
...
Implements https://github.com/astral-sh/ruff/pull/8873 via
https://github.com/astral-sh/ruff/pull/9286
2024-01-04 15:38:42 -06:00
Zanie Blue
967b2dcaf4
Fix ibis ecosystem branch ( #9392 )
2024-01-04 14:18:08 -05:00
Zanie Blue
c01bb0d485
Report errors when repositories cannot be cloned during ecosystem checks ( #9314 )
...
Otherwise, the directory is just missing later and you need to dig
through logs to understand why.
2023-12-29 21:40:22 -06:00
Zanie Blue
fca9dc5ce2
Fix typo in truncation comment ( #9318 )
2023-12-29 21:40:12 -06:00
Zanie Blue
57b6a8cedd
Allow config-file overrides in ecosystem checks ( #9286 )
...
Adds the ability to override `ruff.toml` or `pyproject.toml` settings
per-project during ecosystem checks.
Exploring this as a fix for the `setuptools` project error.
Also useful for including Jupyter Notebooks in the ecosystem checks, see
#9293
Note the remaining `sphinx` project error is resolved in #9294
2023-12-28 10:44:50 -06:00
Zanie Blue
f8fc309855
Exclude failing sphinx file in ecosytem checks ( #9294 )
...
Failing due to
> error: Failed to read tests/roots/test-pycode/cp_1251_coded.py: stream
did not contain valid UTF-8
Unclear to me if ignoring is the correct response.
2023-12-28 10:42:06 -06:00
Zanie Blue
0029b4fd07
Fix ecosystem format line changed counts ( #9158 )
...
We were erroneously including patch headers so for each _file_ changed
we could include an extra added and removed line e.g. we counted the
diff in the following as +4 -4 instead of +3 -3.
```diff
diff --git a/tests/test_param_include_in_schema.py b/tests/test_param_include_in_schema.py
index 26201e9..f461947 100644
--- a/tests/test_param_include_in_schema.py
+++ b/tests/test_param_include_in_schema.py
@@ -9,14 +9,14 @@ app = FastAPI()
@app.get("/hidden_cookie")
async def hidden_cookie(
- hidden_cookie: Optional[str] = Cookie(default=None, include_in_schema=False)
+ hidden_cookie: Optional[str] = Cookie(default=None, include_in_schema=False),
):
return {"hidden_cookie": hidden_cookie}
@app.get("/hidden_header")
async def hidden_header(
- hidden_header: Optional[str] = Header(default=None, include_in_schema=False)
+ hidden_header: Optional[str] = Header(default=None, include_in_schema=False),
):
return {"hidden_header": hidden_header}
@@ -28,7 +28,7 @@ async def hidden_path(hidden_path: str = Path(include_in_schema=False)):
@app.get("/hidden_query")
async def hidden_query(
- hidden_query: Optional[str] = Query(default=None, include_in_schema=False)
+ hidden_query: Optional[str] = Query(default=None, include_in_schema=False),
):
return {"hidden_query": hidden_query}
```
Tested with a single project locally e.g.
> ℹ️ ecosystem check **detected format changes**. (+65 -65 lines in 39
files in 1 projects)
>
> <details><summary><a
href="https://github.com/tiangolo/fastapi ">tiangolo/fastapi</a> (+65 -65
lines across 39 files)
instead of
> ℹ️ ecosystem check **detected format changes**. (+104 -104 lines in 39
files in 1 projects)
>
> <details><summary><a
href="https://github.com/tiangolo/fastapi ">tiangolo/fastapi</a> (+104
-104 lines across 39 files)
2023-12-16 00:14:17 -06:00
Zanie Blue
2c6b534e1f
Update ecosystem check headers to show unchanged project count ( #9157 )
...
Instead of displaying the total completed project count in the "changed"
section of a header, we now separately calculated the changed and
unchanged count to make the header message nice and clear.
e.g.
> ℹ️ ecosystem check **detected format changes**. (+1772 -1859 lines in
239 files in 26 projects; 6 project errors; 9 projects unchanged)
and
> ℹ️ ecosystem check **detected linter changes**. (+4598 -5023
violations, +0 -40 fixes in 13 projects; 4 project errors; 24 projects
unchanged)
Previously, it would have included the unchanged count in the first
project count.
2023-12-16 00:05:38 -06:00
Steve C
3ee1ec70cc
Fix up some types in the ecosystem code ( #8898 )
...
## Summary
Fixes up the type annotations to make type analyzers a little happier 😄
## Test Plan
N/A
2023-11-30 16:02:20 -06:00
T-256
aec80dc3ab
Ruff ecosystem: pass no-preview cli arg by default ( #8775 )
...
Addresses
https://github.com/astral-sh/ruff/pull/8489#issuecomment-1793513411
That issues still exist on formatter, but since `black` doesn't support
`no-preview` cli arg, I didn't include it in this PR.
---------
Co-authored-by: Zanie <contact@zanie.dev>
2023-11-20 12:21:51 -06:00
Zanie Blue
311a7751f9
Ensure ecosystem project errors are properly fenced ( #8516 )
...
Fixes bug where errors could be unfenced resulting in hidden remaining
content
e.g. https://github.com/astral-sh/ruff/pull/8508#issuecomment-1794960132
2023-11-06 09:35:07 -06:00
Zanie Blue
0d93fbb4a2
Only show ecosystem command used if options are non-default ( #8435 )
...
To save that precious character count
2023-11-02 08:53:33 -05:00
Zanie Blue
edc75dc5d6
Pull updates for refs in cached repos in ecosystem checks ( #8420 )
...
Otherwise, the cache can end up not testing the latest changes to the
ref.
2023-11-02 01:30:35 +00:00
