Skip to content

Commit c198a34

Browse files
github-actions[bot]ESLint Bot
andauthored
Add TSC meeting 14-November-2024 transcript (#548)
Co-authored-by: ESLint Bot <eslint[bot]@users.noreply.github.com>
1 parent 5af382a commit c198a34

File tree

1 file changed

+144
-0
lines changed

1 file changed

+144
-0
lines changed

notes/2024/2024-11-14-transcript.md

Lines changed: 144 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,144 @@
1+
# 11/14/2024 ESLint TSC Meeting Transcript
2+
3+
**mdjermanovic:** Hi!
4+
5+
**fasttime:** Hi 👋
6+
7+
**nzakas:** Howdy!
8+
9+
**nzakas:** Looking over last meeting's notes
10+
11+
**nzakas:** Only thing it looks like wasn't accounted for was comparing v9.12 to v9.14 to see what the performance is like. It looks like we neglected to assign that to anyone in particular. Did anyone take a look at that?
12+
13+
**mdjermanovic:** Nope, but we got a response on the original issue that the performance is about the same
14+
15+
**nzakas:** Sounds like a win to me. 👍
16+
* 👍 @mdjermanovic, @fasttime
17+
18+
**mdjermanovic:** (about the same as before v9.12)
19+
20+
**nzakas:** RFC Duty schedule
21+
This week: @nzakas
22+
November 18: @mdjermanovic
23+
November 25: @fasttime
24+
December 2: @nzakas
25+
* 👍 @mdjermanovic, @fasttime
26+
27+
**nzakas:** It looks like we don't have any issues or PRs flagged for today. Are there any that need discussing?
28+
29+
**fasttime:** Nothing in particular from my side.
30+
31+
**mdjermanovic:** Nothing in particular from my side
32+
33+
**nzakas:** One I'd like to discuss: https://github.com/eslint/eslint/issues/3565
34+
35+
**nzakas:** @fasttime mentioned you had a fork of ESLint that does parallel linting. Would you like to create an RFC to add that into the core?
36+
37+
**nzakas:** Ref: https://github.com/eslint/eslint/issues/3565#issuecomment-2406715452
38+
39+
**fasttime:** Definitely! I can prepare a PR and we can clarify the details there. Or I should I prepare an RFC first?
40+
41+
**nzakas:** RFC first 👍
42+
* 👍 @mdjermanovic
43+
44+
**fasttime:** Agreed 👍
45+
46+
**nzakas:** Especially as we seem to keep getting people telling us they're linting repos of 10,000+ files. If we want them to also use ESLint for JSON, Markdown, etc., that number will go up so improving performance is becoming more important.
47+
48+
**nzakas:** Oh, I totally forgot status updates. 😅 Didn't sleep well last night.
49+
50+
**nzakas:** I've been working on the CSS plugin primarily, with a bit of followup work on the JSON plugin, and just started an `extends` RFC.
51+
52+
**fasttime:** Don't tell me about sleeping (father here)...
53+
* 😄 @nzakas
54+
55+
**mdjermanovic:** I was mostly reviewing PRs. Also updated `@eslint/compat` to support legacy `schema` properties.
56+
57+
**fasttime:** I was mostly busy with triaging issues and reviewing RFCs. Also finished the fix on ignoring files on a different drive on Windows
58+
59+
**nzakas:** I was just looking through issues to see anything else jumps out at me. Just a reminder, if you're having trouble getting resolution on an issue, go ahead and label it with 'tsc agenda' so we can over it in this meeting.
60+
* 👍 @mdjermanovic, @fasttime
61+
62+
**nzakas:** I don't see anything pressing at the moment, so shall we talk about the release?
63+
64+
**mdjermanovic:** I can tomorrow
65+
* 🙏 @nzakas
66+
67+
**fasttime:** Thanks!
68+
69+
**mdjermanovic:** That would be `@eslint/eslintrc`, `@eslint/js` and `eslint`
70+
71+
**nzakas:** This will get the `defaultOptions` changes in?
72+
73+
**mdjermanovic:** Yes
74+
* 🎉 @nzakas
75+
76+
**nzakas:** And the `@eslint/plugin-kit` upgrade too?
77+
78+
**fasttime:** There's still a comment to address though
79+
80+
**mdjermanovic:** Yes, here's the PR: https://github.com/eslint/eslint/pull/19130
81+
* 👍 @fasttime
82+
83+
**mdjermanovic:** Which comment?
84+
85+
**nzakas:** Do you mean changing the eslintrc version?
86+
87+
**fasttime:** Ah sorry, I confused it.
88+
89+
**mdjermanovic:** Maybe I didn't close some, but I believe all suggestions have been resolved
90+
91+
**fasttime:** Yes, we can merge that tomorrow.
92+
93+
**nzakas:** Just this one is left: https://github.com/eslint/eslint/pull/17656#discussion_r1820984884
94+
95+
**nzakas:** Which clearly can't happen until `@eslint/eslintrc` is released
96+
97+
**mdjermanovic:** Ah, yes, that needs to wait for the `@eslint/eslintrc` release
98+
99+
**nzakas:** And we'll need to publish the security advisory with the release, as well.
100+
* 👍 @mdjermanovic, @fasttime
101+
102+
**nzakas:** Would probably be helpful to mention in the blog post, too. Maybe not as a "highlight" per se, but an important notice at the top of the post?
103+
* 👍 @mdjermanovic, @fasttime
104+
105+
**nzakas:** Okay, anything else release-related to discuss?
106+
107+
**fasttime:** Any PRs we would like to get merged before tomorrow?
108+
109+
**mdjermanovic:** The one that updates `@eslint/plugin-kit`
110+
111+
**fasttime:** In ESLint we have https://github.com/eslint/eslint/pull/19104 and https://github.com/eslint/eslint/pull/19131, which I think it would be best to release together
112+
* 👍 @nzakas
113+
114+
**fasttime:** Yes, also that one
115+
116+
**nzakas:** Just pushed the missing commit.
117+
118+
**nzakas:** Yeah, I'd like to get those in, too.
119+
* 👍 @mdjermanovic, @fasttime
120+
121+
**nzakas:** This one might be ready: https://github.com/eslint/eslint/pull/19050
122+
123+
**mdjermanovic:** We can release that separately from `eslint`
124+
* 👍 @fasttime
125+
126+
**nzakas:** Good call
127+
128+
**fasttime:** I can review that tomorrow but it would be better to release it separately I think
129+
* 👍 @nzakas, @mdjermanovic
130+
131+
**nzakas:** Anything else?
132+
133+
**fasttime:** Nothing else from me
134+
135+
**mdjermanovic:** Nothing from me too
136+
137+
**nzakas:** Okay, one last thing from me: November 28 is the Thanksgiving holiday in the US, so I'll be off the 28th and the 29th, meaning I'll miss the next TSC meeting.
138+
* 👍 @mdjermanovic, @fasttime
139+
140+
**nzakas:** And that's it for today. Thanks everyone (and thanks @sam3k_ for the notes)
141+
142+
**fasttime:** Thanks!
143+
144+
**mdjermanovic:** Thanks 👋

0 commit comments

Comments
 (0)