|
| 1 | +# 07/24/2025 ESLint TSC Meeting Transcript |
| 2 | + |
| 3 | +**nzakas:** Howdy! |
| 4 | + |
| 5 | +**mdjermanovic:** Hi! |
| 6 | + |
| 7 | +**fasttime:** Hi! |
| 8 | + |
| 9 | +**nzakas:** Grabbing the notes from last meeting |
| 10 | + |
| 11 | +**nzakas:** Okay, nothing to follow up on. |
| 12 | + |
| 13 | +**nzakas:** Let's start with statuses. I was off last week and have mostly been trying to catch up with notifications this week. |
| 14 | + |
| 15 | +**mdjermanovic:** I was switching some tests from eslintrc to flat config mode, in preparation for v10. Also reviewing PRs and RFCs. |
| 16 | + |
| 17 | +**fasttime:** I was mostly busy with triaging and reviews, and worked on some maintenance PRs. |
| 18 | + |
| 19 | +**nzakas:** Let's talk availability over the next couple of weeks. I expect to be around at about 2 hours each week day. |
| 20 | + |
| 21 | +**mdjermanovic:** I expect to work around 1-1.5 hours each day until August 2. Then, I'll be mostly offline Aug 3 - Aug 21 |
| 22 | + |
| 23 | +**fasttime:** I should be available about 9 hours per week |
| 24 | + |
| 25 | +**nzakas:** @mdjermanovic August 2 is next Saturday, so you're around next week and then not again until August 22. |
| 26 | + |
| 27 | +**mdjermanovic:** Yes |
| 28 | + * 👍 @nzakas |
| 29 | + |
| 30 | +**nzakas:** Okay, so we need to figure out RFC duty for the next month. @mdjermanovic was this week. |
| 31 | + |
| 32 | +**mdjermanovic:** I could take the next week too |
| 33 | + * 🙏 @nzakas |
| 34 | + |
| 35 | +**nzakas:** So that would look like: |
| 36 | +This week: @mdjermanovic |
| 37 | +July 28: @mdjermanovic |
| 38 | +August 4: @nzakas |
| 39 | +August 11: @fasttime |
| 40 | + * 👍 @mdjermanovic |
| 41 | + |
| 42 | +**fasttime:** Works for me 👍 |
| 43 | + |
| 44 | +**mdjermanovic:** Works for me too |
| 45 | + |
| 46 | +**nzakas:** Sounds good |
| 47 | + |
| 48 | +**nzakas:** Okay, let's go into the agenda. |
| 49 | + |
| 50 | +**nzakas:** First up: https://github.com/eslint/eslint/issues/19118 |
| 51 | + |
| 52 | +**nzakas:** > **TSC Summary:** Users reported that ESLint's flat config does not match files outside the current working directory, causing issues in monorepo and build tool setups. The discussion led to a proposal for a `--base-path` CLI option, allowing users to explicitly set the root directory for file matching. This approach was seen as a flexible, non-breaking solution that avoids confusing changes to glob pattern semantics. |
| 53 | +> |
| 54 | +> **TSC Question:** Shall we move forward with this solution? |
| 55 | +
|
| 56 | +**mdjermanovic:** I'm in favor |
| 57 | + |
| 58 | +**fasttime:** I'm also in favor |
| 59 | + |
| 60 | +**nzakas:** Okay, we've agreed to move forward with the proposed `--base-path` option. We'll mark this issue as accepted and "Waiting for RFC" |
| 61 | + * 👍 @mdjermanovic, @fasttime |
| 62 | + |
| 63 | +**nzakas:** I'd also like to talk about the parallel linting PR. @fasttime what is the status on getting that ready for review? |
| 64 | + |
| 65 | +**fasttime:** Should be ready next week |
| 66 | + * 🎉 @michael.faith, @nzakas, @mdjermanovic |
| 67 | + |
| 68 | +**nzakas:** Can you prioritize working on that so we can take advantage of @mdjermanovic's last week to get his review? |
| 69 | + |
| 70 | +**fasttime:** Sure, I'm planning to implement the last todo on the weekend, so hopefully Monday or Tuedsay it should be ready. |
| 71 | + * 🎉 @mdjermanovic |
| 72 | + * 👍 @nzakas |
| 73 | + |
| 74 | +**nzakas:** Okay, any other issues or PRs to discuss before we talk about v10? |
| 75 | + |
| 76 | +**mdjermanovic:** Nothing in particular from me for today |
| 77 | + |
| 78 | +**fasttime:** Nothing in particular from my end |
| 79 | + |
| 80 | +**nzakas:** Here's the v10 board: https://github.com/orgs/eslint/projects/6' |
| 81 | + |
| 82 | +**nzakas:** First question: is there anything anyone has been thinking about for v10 that isn't already represented on the board? |
| 83 | + |
| 84 | +**mdjermanovic:** Maybe this one: https://github.com/eslint/js/issues/665 |
| 85 | + |
| 86 | +**fasttime:** I was hoping to see async parsers and rules. |
| 87 | + |
| 88 | +**nzakas:** We can't do that without the core rewrite, and that won't happen until after v10. |
| 89 | + * 👍 @fasttime |
| 90 | + |
| 91 | +**mdjermanovic:** Also not sure if making the `v10_config_lookup_from_file` default behavior is included in any of the issues we have in the v10 project. |
| 92 | + * 🤔 @nzakas |
| 93 | + |
| 94 | +**fasttime:** Oh, and extracting core rules to `@eslint/js` is also planned for v10 |
| 95 | + |
| 96 | +**nzakas:** Good catch, we should probably open an issue to flip all `v10_` flags to "on" |
| 97 | + * 👍 @mdjermanovic |
| 98 | + |
| 99 | +**nzakas:** I don't recall discussing that for v10. Is there an issue somewhere? |
| 100 | + |
| 101 | +**nzakas:** (We've definitely discussed it "in the future",) |
| 102 | + |
| 103 | +**mdjermanovic:** I think it's included in https://github.com/eslint/eslint/issues/19013 |
| 104 | + |
| 105 | +**nzakas:** Ah ok, and that's already on the board. |
| 106 | + |
| 107 | +**fasttime:** Thanks, yes, I remembered the title was not directly related. |
| 108 | + |
| 109 | +**nzakas:** I'll propose we defer that. The amount of infrastructure work alone would probably end up pushing that into next year, and I'd prefer not to wait that long for v10. |
| 110 | + |
| 111 | +**mdjermanovic:** I was also thinking we could postpone that |
| 112 | + |
| 113 | +**fasttime:** This one should be also deferred if we want to be ready in the next few months: https://github.com/eslint/eslint/issues/17579 |
| 114 | + * 👍 @nzakas, @mdjermanovic |
| 115 | + |
| 116 | +**fasttime:** I haven't been following up on that because of other topics |
| 117 | + |
| 118 | +**nzakas:** Let me create a new v11 board to move these to |
| 119 | + * 👍 @mdjermanovic, @fasttime |
| 120 | + |
| 121 | +**nzakas:** Okay, moved those. |
| 122 | + |
| 123 | +**nzakas:** And added https://github.com/eslint/js/issues/665 to the v10 board |
| 124 | + * 👍 @mdjermanovic, @fasttime |
| 125 | + |
| 126 | +**nzakas:** Okay and here's the issue for `v10_` flags: https://github.com/eslint/eslint/issues/19967 |
| 127 | + * 👍 @mdjermanovic |
| 128 | + |
| 129 | +**nzakas:** Looks like we only have one...which surprises me for some reason. 🤷♂️ |
| 130 | + |
| 131 | +**nzakas:** Do we want to go through each of the issues on the board and give it a 👍 or 👎? |
| 132 | + * 👍 @mdjermanovic, @fasttime |
| 133 | + |
| 134 | +**nzakas:** Finish Language plugins cleanup: |
| 135 | +https://github.com/eslint/eslint/issues/16999 |
| 136 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 137 | + |
| 138 | +**nzakas:** Remove `globalReturn`: |
| 139 | +https://github.com/eslint/js/issues/525 |
| 140 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 141 | + |
| 142 | +**nzakas:** Finish Flat Config/Remove eslintrc: |
| 143 | +https://github.com/eslint/eslint/issues/13481 |
| 144 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 145 | + |
| 146 | +**nzakas:** Stop using lodash packages: |
| 147 | +https://github.com/eslint/eslint/issues/18160 |
| 148 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 149 | + |
| 150 | +**nzakas:** Update minimatch: |
| 151 | +https://github.com/eslint/rewrite/issues/66 |
| 152 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 153 | + |
| 154 | +**nzakas:** no-invalid-regexp option fix: |
| 155 | +https://github.com/eslint/eslint/issues/18755 |
| 156 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 157 | + |
| 158 | +**nzakas:** fixer methods throw errors for non-string arguments: |
| 159 | +https://github.com/eslint/eslint/issues/18807 |
| 160 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 161 | + |
| 162 | +**nzakas:** Make RuleTester stricter: |
| 163 | +https://github.com/eslint/eslint/issues/18960 |
| 164 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 165 | + |
| 166 | +**mdjermanovic:** That one has several proposals. I think we have a consensus only on the first point. |
| 167 | + |
| 168 | +**nzakas:** That's okay. The question is do we want to plan on doing this for v10? If yes, we can work on the details. |
| 169 | + |
| 170 | +**mdjermanovic:** Okay, gave 👍 |
| 171 | + |
| 172 | +**nzakas:** Don't pass BOM to preprocess method: |
| 173 | +https://github.com/eslint/eslint/issues/18891 |
| 174 | + * 👍 @mdjermanovic, @fasttime |
| 175 | + * 👎 @nzakas |
| 176 | + |
| 177 | +**nzakas:** I know I filed this, but I'd like to push it to v11. I feel like there might be some yucky edge cases to dig into. |
| 178 | + |
| 179 | +**mdjermanovic:** Sounds good to me |
| 180 | + |
| 181 | +**fasttime:** Okay |
| 182 | + |
| 183 | +**nzakas:** Remove message.nodeType/testErrorCase.type: |
| 184 | +https://github.com/eslint/eslint/issues/19029 |
| 185 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 186 | + |
| 187 | +**nzakas:** Update no-shadow-restricted-names to report globalThis by default: |
| 188 | +https://github.com/eslint/eslint/issues/19673 |
| 189 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 190 | + |
| 191 | +**nzakas:** Remove support for jiti < 2.2.0: |
| 192 | +https://github.com/eslint/eslint/issues/19765 |
| 193 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 194 | + |
| 195 | +**nzakas:** Change `Program` range to cover full file: |
| 196 | +https://github.com/eslint/js/issues/648 |
| 197 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 198 | + |
| 199 | +**nzakas:** Add back `name` to recommended configs: |
| 200 | +https://github.com/eslint/eslint/issues/19864 |
| 201 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 202 | + |
| 203 | +**nzakas:** Update `eslint:recommended`: |
| 204 | +https://github.com/eslint/eslint/issues/19966 |
| 205 | + * 👍 @nzakas, @mdjermanovic, @fasttime |
| 206 | + |
| 207 | +**nzakas:** Okay, that is it! |
| 208 | + |
| 209 | +**mdjermanovic:** We'll also have an issue for dropping support for old versions of Node.js |
| 210 | + |
| 211 | +**nzakas:** Ooh good catch. Let me make that right now. |
| 212 | + * 👍 @mdjermanovic, @fasttime |
| 213 | + |
| 214 | +**nzakas:** https://github.com/eslint/eslint/issues/19969 |
| 215 | + |
| 216 | +**nzakas:** Added to the board. |
| 217 | + * 👍 @mdjermanovic, @fasttime |
| 218 | + |
| 219 | +**nzakas:** Okay, NOW I think we have it? |
| 220 | + |
| 221 | +**mdjermanovic:** I think that's all |
| 222 | + |
| 223 | +**fasttime:** I think so |
| 224 | + |
| 225 | +**nzakas:** All right, we'll operate on the assumption that we've just defined v10 unless anything else pressing comes up. Hopefully the scope will be a lot easier to manage than v9. |
| 226 | + * 👍 @mdjermanovic, @fasttime |
| 227 | + |
| 228 | +**nzakas:** That's a good transition to talking about the release tomorrow. 😄 |
| 229 | + |
| 230 | +**nzakas:** I'll take that silence as no one is available. I can do it. |
| 231 | + |
| 232 | +**mdjermanovic:** Thanks! |
| 233 | + |
| 234 | +**fasttime:** Thanks! |
| 235 | + |
| 236 | +**nzakas:** I think that's just `eslint` and `@eslint/js`? |
| 237 | + * 👍 @mdjermanovic, @fasttime |
| 238 | + |
| 239 | +**nzakas:** Any PRs anyone is looking to get in? |
| 240 | + |
| 241 | +**mdjermanovic:** This one has two approvals but few open questions: https://github.com/eslint/eslint/pull/19877 |
| 242 | + |
| 243 | +**nzakas:** Thanks, still digging through notifications. I'll reply to those. |
| 244 | + * 👍 @mdjermanovic, @fasttime |
| 245 | + |
| 246 | +**mdjermanovic:** Nothing else looks definitely ready to me. |
| 247 | + |
| 248 | +**nzakas:** Yeah, I don't see anything else. I'll re-review before the release just to double-check. |
| 249 | + |
| 250 | +**nzakas:** Okay, that's all for today. Thanks everyone! (And thanks @sam3k_ 3k for the notes) |
| 251 | + |
| 252 | +**fasttime:** Thanks! |
| 253 | + |
| 254 | +**mdjermanovic:** Thanks 👋 |
0 commit comments