Skip to content

Commit be75114

Browse files
github-actions[bot]ESLint Bot
andauthored
Add TSC meeting 15-May-2025 transcript (#586)
Co-authored-by: ESLint Bot <eslint[bot]@users.noreply.github.com>
1 parent 1e372b2 commit be75114

File tree

1 file changed

+115
-0
lines changed

1 file changed

+115
-0
lines changed

notes/2025/2025-05-15-transcript.md

Lines changed: 115 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,115 @@
1+
# 05/15/2025 ESLint TSC Meeting Transcript
2+
3+
**nzakas:** Howdy!
4+
5+
**mdjermanovic:** Hi!
6+
7+
**fasttime:** Hi there!
8+
9+
**nzakas:** Just looking up notes from last time
10+
11+
**nzakas:** Looks like we didn't have any action items other than following up open RFCs, which we did.
12+
13+
**nzakas:** Let's start with statuses. I've been continuing to refactor the core, bug fixing in CSSTree, and externalizing the MCP server.
14+
15+
**mdjermanovic:** I was reviewing a lot of PRs.
16+
17+
**fasttime:** I have fixed some issues with the types and started working on a prototype for multithread linting.
18+
19+
**nzakas:** Speaking of RFCs, here's the upcoming RFC duty schedule:
20+
This week - @fasttime
21+
May 19 - @nzakas
22+
May 26 - @mdjermanovic
23+
* 👍 @mdjermanovic, @fasttime
24+
25+
**nzakas:** Let's talk availability for the next couple of weeks. I expect to have 3 hours each day, outside of May 26, which is a holiday in the US.
26+
27+
**mdjermanovic:** I expect to have around 2 hours each day.
28+
29+
**fasttime:** I also expect to be available 2 hours a day the next two weeks
30+
31+
**nzakas:** Great, let's dig in. I flagged a few things to discuss.
32+
33+
**nzakas:** First is the `basePath` RFC: https://github.com/eslint/rfcs/pull/131
34+
35+
**nzakas:** > Because this issue of `basePath` in `extends` is the only thing left to resolve, let's go back to @mdjermanovic's original suggestion:
36+
>
37+
> > to avoid confusion in expectations, defineConfig() should throw an error if it encounters basePath in extended configs?
38+
>
39+
> I'm okay with this, as it allows us the opportunity to think through this use case further and make changes down the line if necessary. I think it's important to get this change out relatively soon so we can start v10 planning.
40+
41+
**mdjermanovic:** I'm in favor of my original suggestion. Combining configs with base paths seems complicated. Throwing an error allows us to think through this further and eventually support this.
42+
43+
**mdjermanovic:** As adding a behavior for something that used to throw an error is generally considered a non-breaking change.
44+
* 👍 @nzakas
45+
46+
**fasttime:** I'm in favor of throwing an error instead of combining `basePath`s for a start. I'd like to read the RFC once again before approving.
47+
48+
**mdjermanovic:** I need to update the RFC with details about `defineConfig()` anyway
49+
50+
**nzakas:** Okay, we've agreed to throw an error when there's `basePath` in `extends`. Can we also agree to prioritize updating and approving this RFC?
51+
* 👍 @mdjermanovic, @fasttime
52+
53+
**nzakas:** Great, looking forward to getting the v10 train moving. 😄
54+
55+
**nzakas:** Next item: https://github.com/eslint/eslint/issues/19719
56+
57+
**nzakas:** > The [`func-style`](https://eslint.org/docs/latest/rules/func-style) rule is currently frozen, meaning we aren't making any further changes to it and we'd need the [@eslint/eslint-tsc](https://github.com/orgs/eslint/teams/eslint-tsc) to override that. I believe the intent here is to make `func-style` TypeScript-aware, which seems like it goes along with our ongoing work to make ESLint as a whole more TypeScript-aware. As a result, I'm 👍 to overriding the frozen status of this rule for this particular purpose.
58+
59+
**mdjermanovic:** If the question is does making core rules TypeScript-aware override frozen status, I'd say yes when a rule is buggy in TS code or doesn't seem to support certain aspect of TS syntax well.
60+
61+
**mdjermanovic:** I think that's in line with our policies about frozen rules. That is that we fix bugs and support new syntax. TypeScript syntax is kind of "new" since we recently decided to support it.
62+
* 👍 @nzakas
63+
64+
**mdjermanovic:** As for this particular request, isn't it the same as this one, which we didn't accept? https://github.com/eslint/eslint/issues/18842
65+
66+
**nzakas:** Yes. We declined to implement at that time because we had not yet decided how to handle TypeScript-specific syntax in core rules.
67+
68+
**mdjermanovic:** Okay, I'm in favor of this change, and adding this behavior behind an option as suggested seems like a good compromise
69+
70+
**nzakas:** @fasttime already 👍ed on the issue, but just want to double check
71+
72+
**nzakas:** (for the record)
73+
74+
**fasttime:** Sounds good to me. Let's do it.
75+
76+
**nzakas:** Okay, we've agreed to implement this change behind a new option. We've also agreed that updating a frozen rule to support TypeScript syntax makes sense.
77+
78+
I'll take the action to update the description of frozen rules to make that explicit.
79+
* 👍 @mdjermanovic
80+
81+
**fasttime:** Thanks! That's a good idea. So the team will know how to handle such requests next time.
82+
83+
**nzakas:** Next item:
84+
> Do we want to post release messages in the Discord <#710284269657784440> channel? We can automate that along with releasing to social media.
85+
86+
**nzakas:** Release messages for things like `@eslint/css`, etc.
87+
88+
**mdjermanovic:** I'm in favor
89+
90+
**fasttime:** I'm also in favor.
91+
92+
**nzakas:** Okay, we've agreed to automatically post release messages to <#710284269657784440>. I'll take the action to set that up.
93+
* 👍 @mdjermanovic, @fasttime
94+
95+
**nzakas:** Those are all I had flagged for today. Any other topics anyone would like to discuss?
96+
97+
**mdjermanovic:** Nothing in particular from my side.
98+
99+
**fasttime:** Nothing from my side.
100+
101+
**nzakas:** All right, then let's talk about the release.
102+
103+
**mdjermanovic:** I can tomorrow
104+
* 🙏 @nzakas
105+
106+
**fasttime:** Thanks!
107+
108+
**mdjermanovic:** That would be only `eslint` and `@eslint/js` this time, I believe.
109+
* 👍 @nzakas, @fasttime
110+
111+
**nzakas:** All right, if there's nothing else, then I think we're done. Thanks everyone (and thanks @sam3k_ for the notes)
112+
113+
**fasttime:** Thanks!
114+
115+
**mdjermanovic:** Thanks! 👋

0 commit comments

Comments
 (0)