|
| 1 | +# 08/07/2025 ESLint TSC Meeting Transcript |
| 2 | + |
| 3 | +**nzakas:** Howdy! |
| 4 | + |
| 5 | +**fasttime:** Hi! |
| 6 | + |
| 7 | +**nzakas:** It'll just be us today as @mdjermanovic is away |
| 8 | + |
| 9 | +**fasttime:** Yes |
| 10 | + |
| 11 | +**nzakas:** Just going through last meeting's notes |
| 12 | + |
| 13 | +**nzakas:** Looks like we don't have any followups from last week. |
| 14 | + |
| 15 | +**nzakas:** Last meeting, that is. |
| 16 | + * 👍 @fasttime |
| 17 | + |
| 18 | +**nzakas:** Let's start with statuses. I've mostly been reviewing PRs for the Markdown and CSS plugins, which have had a lot of activity. I updated the MCP server prompt so it allows autofixing if requested, and continued work on the CSS variables RFC. |
| 19 | + |
| 20 | +**fasttime:** I've been mostly busy reviewing PRs and a few issues. Other than that, I've updated the multithread linting PR and worked on some maintenance tasks. |
| 21 | + |
| 22 | +**nzakas:** Let's talk availability for the next couple of weeks. I'm going to have a bit less time, around one hour per weekday, until the end of August. |
| 23 | + |
| 24 | +**fasttime:** I should be able to work about 1 hour during weekdays and maybe 2 on the weekends, so like 9 hours per week. I expect to have more time starting August 18. |
| 25 | + |
| 26 | +**nzakas:** RFC Duty: |
| 27 | +This week - @nzakas |
| 28 | +August 11 - @fasttime |
| 29 | +August 18 - @nzakas |
| 30 | + * 👍 @fasttime |
| 31 | + |
| 32 | +**nzakas:** It doesn't look like we have anything flagged for today. Any issues or PRs you'd like to discuss? |
| 33 | + |
| 34 | +**fasttime:** Nothing in particular. |
| 35 | + |
| 36 | +**nzakas:** Want to share a status update on parallel linting? |
| 37 | + |
| 38 | +**fasttime:** There's only one open question: https://github.com/eslint/eslint/pull/19794#discussion_r2235619344 |
| 39 | + |
| 40 | +**fasttime:** Other than that no feedback to address. Shall we write a blog post? |
| 41 | + |
| 42 | +**nzakas:** Write a blog about...? |
| 43 | + |
| 44 | +**fasttime:** About the multithread linting, I mean |
| 45 | + |
| 46 | +**nzakas:** When it's released, yes. |
| 47 | + |
| 48 | +**fasttime:** Okay 🙂 That's all I think. |
| 49 | + |
| 50 | +**nzakas:** For including `URL`, it looks like it's available in the TypeScript DOM library, so we could do: |
| 51 | + |
| 52 | +```json |
| 53 | +{ |
| 54 | + "compilerOptions": { |
| 55 | + "lib": ["ESNext", "DOM"] |
| 56 | + } |
| 57 | +} |
| 58 | +``` |
| 59 | + |
| 60 | +? |
| 61 | + |
| 62 | +**nzakas:** Although we don't have a `tsconfig.json` at the top-level. 🤔 |
| 63 | + |
| 64 | +**fasttime:** I think the problem is more consumers that don't use the DOM library or `@types/node` in their dependencies. |
| 65 | + |
| 66 | +**fasttime:** For that we should add a production dependency on `@types/node` or ask TS consumers to use `@types/node` or a built-in library that provides a definition for `URL`. |
| 67 | + |
| 68 | +**nzakas:** That feels pretty clunky |
| 69 | + |
| 70 | +**fasttime:** Yes |
| 71 | + |
| 72 | +**nzakas:** Because this is the last blocking decision, then let's go with your idea of a custom type for now. As long as we have a runtime check for instanceof URL, then I think that gets us most of what we want. |
| 73 | + |
| 74 | +**fasttime:** Sounds good. We can probably change this later with a simple fix. |
| 75 | + |
| 76 | +**nzakas:** Yeah, that's my thinking too. |
| 77 | + |
| 78 | +**fasttime:** Okay, I'll do the change. |
| 79 | + * 👍 @nzakas |
| 80 | + |
| 81 | +**nzakas:** Oh, and OptionModule vs. OptionsModule? |
| 82 | + |
| 83 | +**nzakas:** Ah looks like you already made that change |
| 84 | + |
| 85 | +**nzakas:** Sorry, haven't caught up yet today. |
| 86 | + |
| 87 | +**fasttime:** Yeah 🙂 I did that already. |
| 88 | + * 👍 @nzakas |
| 89 | + |
| 90 | +**nzakas:** Okay, anything else before we talk contributor pool? |
| 91 | + |
| 92 | +**fasttime:** Nothing from my side. |
| 93 | + |
| 94 | +**nzakas:** Here's the report from July: |
| 95 | +https://github.com/eslint/tsc-meetings/blob/main/notes/2025/2025-07-01-contributor-pool.md |
| 96 | + |
| 97 | +**nzakas:** Another massive month for contributions. |
| 98 | + * 😮 @fasttime |
| 99 | + |
| 100 | +**nzakas:** I'm going to ask ChatGPT to review and give some recommendations. 🙂 |
| 101 | + |
| 102 | +**fasttime:** Good idea, let's give it a try. |
| 103 | + |
| 104 | +**nzakas:** Here's what I got: |
| 105 | + |
| 106 | +# 💰 Contributor Payouts – June 2025 |
| 107 | + |
| 108 | +Based on PR count, merge time, and interaction. |
| 109 | + |
| 110 | +## Top Tier |
| 111 | +**xbinaryx** (7 PRs) – Major features, long reviews, high activity |
| 112 | +**→ $700** |
| 113 | + |
| 114 | +## Strong Contributors |
| 115 | +**sethamus** (4 PRs) – Core ESLint features, solid feedback |
| 116 | +**→ $450** |
| 117 | +**TKDev7** (3 PRs) – Fast merges, steady work |
| 118 | +**→ $300** |
| 119 | + |
| 120 | +## Focused Feature Work |
| 121 | +**SwetaTanwar** (2 PRs) – Original rules, detailed reviews |
| 122 | +**→ $250** |
| 123 | +**Pixel998** (2 PRs) – JSON frontmatter in two repos |
| 124 | +**→ $200** |
| 125 | + |
| 126 | +## Impactful One-Offs |
| 127 | +**jgoz** (1 PR) – Heavy discussion |
| 128 | +**→ $200** |
| 129 | +**remcohaszing** (1 PR) – Longest merge delay |
| 130 | +**→ $200** |
| 131 | +**fisker** (1 PR) – ES2025 globals |
| 132 | +**→ $100** |
| 133 | +**ota-meshi** (1 PR) – ES2026 `using`, quick merge |
| 134 | +**→ $100** |
| 135 | + |
| 136 | +--- |
| 137 | + |
| 138 | +### 📊 Summary |
| 139 | + |
| 140 | +| Contributor | PRs | 💵 | |
| 141 | +|----------------|-----|-----| |
| 142 | +| xbinaryx | 7 | $700 | |
| 143 | +| sethamus | 4 | $450 | |
| 144 | +| TKDev7 | 3 | $300 | |
| 145 | +| SwetaTanwar | 2 | $250 | |
| 146 | +| Pixel998 | 2 | $200 | |
| 147 | +| jgoz | 1 | $200 | |
| 148 | +| remcohaszing | 1 | $200 | |
| 149 | +| fisker | 1 | $100 | |
| 150 | +| ota-meshi | 1 | $100 | |
| 151 | + |
| 152 | +**Total: $2,500** |
| 153 | + |
| 154 | +**nzakas:** I think I'd probably bump xbinaryx up to $800 |
| 155 | + |
| 156 | +**nzakas:** Removing rollup and extra types from `@eslint/markdown` was a larger job |
| 157 | + |
| 158 | +**fasttime:** I didn't review their PRs but I also think that some of them seem larger |
| 159 | + |
| 160 | +**nzakas:** Ah some of these are for new rules, which we typically pay higher for, let me adjust that algorithm and see what we come up with. |
| 161 | + |
| 162 | +**nzakas:** This looks better |
| 163 | + |
| 164 | +# 💰 Contributor Payouts – June 2025 |
| 165 | + |
| 166 | +Includes bonus for new rules and breaking changes. |
| 167 | + |
| 168 | +## Top Tier |
| 169 | +**xbinaryx** (7 PRs) |
| 170 | +- 4 new rules (🧱), 1 breaking change (🔥), 11-comment thread |
| 171 | +**→ $950** |
| 172 | + |
| 173 | +## Strong Contributors |
| 174 | +**sethamus** (4 PRs) – Feature options, steady value |
| 175 | +**→ $400** |
| 176 | +**TKDev7** (3 PRs) – Fixes + config additions, fast merges |
| 177 | +**→ $300** |
| 178 | + |
| 179 | +## Feature-Focused |
| 180 | +**SwetaTanwar** (2 PRs) |
| 181 | +- 2 new rules (🧱), long review cycle |
| 182 | +**→ $350** |
| 183 | +**Pixel998** (2 PRs) – JSON frontmatter support |
| 184 | +**→ $200** |
| 185 | + |
| 186 | +## High-Impact Singles |
| 187 | +**jgoz** (1 PR) – Option addition with 11 comments |
| 188 | +**→ $200** |
| 189 | +**remcohaszing** (1 PR) – Longest delay + solid discussion |
| 190 | +**→ $200** |
| 191 | + |
| 192 | +## Quick & Clean |
| 193 | +**fisker** (1 PR) – ES2025 globals |
| 194 | +**→ $100** |
| 195 | +**ota-meshi** (1 PR) – ES2026 `using` support |
| 196 | +**→ $100** |
| 197 | + |
| 198 | +--- |
| 199 | + |
| 200 | +### 📊 Summary |
| 201 | + |
| 202 | +| Contributor | PRs | 💵 | |
| 203 | +|----------------|-----|-----| |
| 204 | +| xbinaryx | 7 | $950 | |
| 205 | +| sethamus | 4 | $400 | |
| 206 | +| TKDev7 | 3 | $300 | |
| 207 | +| SwetaTanwar | 2 | $350 | |
| 208 | +| Pixel998 | 2 | $200 | |
| 209 | +| jgoz | 1 | $200 | |
| 210 | +| remcohaszing | 1 | $200 | |
| 211 | +| fisker | 1 | $100 | |
| 212 | +| ota-meshi | 1 | $100 | |
| 213 | + |
| 214 | +**Total: $2,800** |
| 215 | + |
| 216 | +**fasttime:** What was your prompt? |
| 217 | + |
| 218 | +**nzakas:** I'll share it with you outside of the meeting |
| 219 | + * 👍 @fasttime |
| 220 | + |
| 221 | +**nzakas:** Thoughts on this? |
| 222 | + |
| 223 | +**fasttime:** I'd round up xbinaryx's amount to $1000 |
| 224 | + * 👍 @nzakas |
| 225 | + |
| 226 | +**fasttime:** This PR was merged in June: https://github.com/eslint/js/pull/658 |
| 227 | + |
| 228 | +**nzakas:** Hmm, that's strange. |
| 229 | + |
| 230 | +**fasttime:** I thought we rewarded that already? |
| 231 | + |
| 232 | +**nzakas:** Fortunately, we now have reports to refer back to |
| 233 | + |
| 234 | +**nzakas:** It wasn't on last month's report. 🤔 |
| 235 | + |
| 236 | +**nzakas:** Let me check the notes |
| 237 | + |
| 238 | +**nzakas:** We did pay ota-meshi for this one |
| 239 | + |
| 240 | +**fasttime:** The dates are 06/01/2025 - 06/30/2025 |
| 241 | + |
| 242 | +**nzakas:** Ah crap, I think I opened the wrong file. 😆 |
| 243 | + |
| 244 | +**fasttime:** I don't see a new one. |
| 245 | + |
| 246 | +**nzakas:** Forgot to merge the PR. |
| 247 | +https://github.com/eslint/tsc-meetings/pull/607 |
| 248 | + * 😅 @fasttime |
| 249 | + |
| 250 | +**nzakas:** Here we go: https://github.com/eslint/tsc-meetings/blob/main/notes/2025/2025-08-01-contributor-pool.md |
| 251 | + |
| 252 | +**nzakas:** I'll run this through ChatGPT and see what we come up with. |
| 253 | + |
| 254 | +**nzakas:** # 💰 Contributor Payouts – July 2025 |
| 255 | + |
| 256 | +## Top Tier |
| 257 | +**TKDev7** (6 PRs) |
| 258 | +- 2 config features, 1 new rule (🧱), 1 breaking change (💥), useful fixes |
| 259 | +**→ $700** |
| 260 | + |
| 261 | +## Strong Contributors |
| 262 | +**xbinaryx** (6 PRs) |
| 263 | +- 2 options, 1 config automation, 1 long-review feature |
| 264 | +**→ $500** |
| 265 | + |
| 266 | +**DMartens** (3 PRs) |
| 267 | +- 2 notable features (long merges), 1 fix |
| 268 | +**→ $350** |
| 269 | + |
| 270 | +## Feature Contributors |
| 271 | +**thecalamiity** (2 PRs) |
| 272 | +- 1 new rule (🧱), 1 enhancement |
| 273 | +**→ $275** |
| 274 | + |
| 275 | +**SwetaTanwar** (1 PR) |
| 276 | +- Core rule feature addition |
| 277 | +**→ $175** |
| 278 | + |
| 279 | +**sethamus** (1 PR) |
| 280 | +- Option added to core rule, solid merge time |
| 281 | +**→ $150** |
| 282 | + |
| 283 | +**ST-DDT** (1 PR) |
| 284 | +- High interaction (14 💬), core testing improvement |
| 285 | +**→ $175** |
| 286 | + |
| 287 | +## Fix-Oriented |
| 288 | +**ericcornelissen** (2 PRs) – Regex performance fixes |
| 289 | +**→ $200** |
| 290 | +**andreww2012** (1 PR) – Small but useful fix |
| 291 | +**→ $100** |
| 292 | + |
| 293 | +--- |
| 294 | + |
| 295 | +### 📊 Summary |
| 296 | + |
| 297 | +| Contributor | PRs | 💵 | |
| 298 | +|----------------|-----|-----| |
| 299 | +| TKDev7 | 6 | $700 | |
| 300 | +| xbinaryx | 6 | $500 | |
| 301 | +| DMartens | 3 | $350 | |
| 302 | +| thecalamiity | 2 | $275 | |
| 303 | +| SwetaTanwar | 1 | $175 | |
| 304 | +| sethamus | 1 | $150 | |
| 305 | +| ST-DDT | 1 | $175 | |
| 306 | +| ericcornelissen| 2 | $200 | |
| 307 | +| andreww2012 | 1 | $100 | |
| 308 | + |
| 309 | +**Total: $2,625** |
| 310 | + |
| 311 | +**fasttime:** Okay, that looks correct. |
| 312 | + |
| 313 | +**nzakas:** I think I would bump DMartens up to $500. The core rules work was a lot. |
| 314 | + |
| 315 | +**fasttime:** Yes, definitely |
| 316 | + |
| 317 | +**nzakas:** And xbinaryx to $700. |
| 318 | + * 👍 @fasttime |
| 319 | + |
| 320 | +**nzakas:** The rest look roughly correct to me. |
| 321 | + |
| 322 | +**fasttime:** Okay |
| 323 | + |
| 324 | +**nzakas:** Okay, I'll let them know. |
| 325 | + * 👍 @fasttime |
| 326 | + |
| 327 | +**nzakas:** For easier review: |
| 328 | + |
| 329 | +| Contributor | PRs | 💵 | |
| 330 | +|----------------|-----|-----| |
| 331 | +| TKDev7 | 6 | $700 | |
| 332 | +| xbinaryx | 6 | $700 | |
| 333 | +| DMartens | 3 | $500 | |
| 334 | +| thecalamiity | 2 | $275 | |
| 335 | +| SwetaTanwar | 1 | $175 | |
| 336 | +| sethamus | 1 | $150 | |
| 337 | +| ST-DDT | 1 | $175 | |
| 338 | +| ericcornelissen| 2 | $200 | |
| 339 | +| andreww2012 | 1 | $100 | |
| 340 | + * 👍 @fasttime |
| 341 | + |
| 342 | +**nzakas:** Let's talk the release tomorrow. |
| 343 | + |
| 344 | +**fasttime:** I can do the release tomorrow. |
| 345 | + |
| 346 | +**nzakas:** Thanks! |
| 347 | + |
| 348 | +**fasttime:** There are some PRs pending a second review, I'm not sure if there's anything important for tomorrows release. |
| 349 | + |
| 350 | +**nzakas:** We should probably merge this and bump `@eslint/core`: https://github.com/eslint/rewrite/pull/252 |
| 351 | + |
| 352 | +**fasttime:** Okay, I can do that tomorrow. |
| 353 | + * 👍 @nzakas |
| 354 | + |
| 355 | +**fasttime:** Other than that it will be only `eslint` and `@eslint/js` I think. |
| 356 | + |
| 357 | +**nzakas:** Looks like it |
| 358 | + * 👍 @fasttime |
| 359 | + |
| 360 | +**nzakas:** If there's not anything else, then I think we're done. Thanks (and thanks @sam3k_ for the notes) |
| 361 | + |
| 362 | +**fasttime:** Thanks! Bye 👋 |
| 363 | + |
| 364 | +**nzakas:** 👋 |
0 commit comments