Skip to content

Commit 9a676ee

Browse files
authored
Fix rust compiler meeting info (#1087)
* Remove dead link * Make general text a bit better * Fix Zulip link * Improve description of what the compiler meeting is about
1 parent 67ebd4b commit 9a676ee

File tree

1 file changed

+24
-18
lines changed

1 file changed

+24
-18
lines changed

src/compiler-team.md

+24-18
Original file line numberDiff line numberDiff line change
@@ -30,32 +30,38 @@ who are experts on each one.
3030
## Rust compiler meeting
3131

3232
The compiler team has a weekly meeting where we do triage and try to
33-
generally stay on top of new bugs, regressions, and other things.
34-
They are held on [Zulip][zulip]. It works roughly as follows:
35-
36-
- **Review P-high bugs:** P-high bugs are those that are sufficiently
37-
important for us to actively track progress. P-high bugs should
38-
ideally always have an assignee.
39-
- **Look over new regressions:** we then look for new cases where the
40-
compiler broke previously working code in the wild. Regressions are
41-
almost always marked as P-high; the major exception would be bug
42-
fixes (though even there we often [aim to give warnings first][procedure]).
43-
- **Check I-nominated issues:** These are issues where feedback from
33+
generally stay on top of new bugs, regressions, and discuss important
34+
things in general.
35+
They are held on [Zulip][zulip-meetings]. It works roughly as follows:
36+
37+
- **Announcements, MCPs/FCPs, and WG-check-ins:** We share some
38+
announcements with the rest of the team about important things we want
39+
everyone to be aware of. We also share the status of MCPs and FCPs and we
40+
use the opportunity to have a couple of WGs giving us an update about
41+
their work.
42+
- **Check for beta and stable nominations:** These are nominations of things to
43+
backport to beta and stable respectively.
44+
We then look for new cases where the compiler broke previously working
45+
code in the wild. Regressions are important issues to fix, so it's
46+
likely that they are tagged as P-critical or P-high; the major
47+
exception would be bug fixes (though even there we often [aim to give
48+
warnings first][procedure]).
49+
- **Review P-critical and P-high bugs:** P-critical and P-high bugs are
50+
those that are sufficiently important for us to actively track
51+
progress. P-critical and P-high bugs should ideally always have an
52+
assignee.
53+
- **Check S-waiting-on-team and I-nominated issues:** These are issues where feedback from
4454
the team is desired.
45-
- **Check for beta nominations:** These are nominations of things to
46-
backport to beta.
47-
- **Possibly WG checking:** A WG may give an update at this point, if there is
48-
time.
55+
- **Look over the performance triage report:** We check for PRs that made the
56+
performance worse and try to decide if it's worth reverting the performance regression or if
57+
the regression can be addressed in a future PR.
4958

5059
The meeting currently takes place on Thursdays at 10am Boston time
5160
(UTC-4 typically, but daylight savings time sometimes makes things
5261
complicated).
5362

54-
The meeting is held over a "chat medium", currently on [zulip].
5563

56-
[etherpad]: https://public.etherpad-mozilla.org/p/rust-compiler-meeting
5764
[procedure]: ./bug-fix-procedure.md
58-
[zulip]: https://rust-lang.zulipchat.com/#narrow/stream/131828-t-compiler
5965
[zulip-help]: https://rust-lang.zulipchat.com/#narrow/stream/182449-t-compiler.2Fhelp
6066
[zulip-meetings]: https://rust-lang.zulipchat.com/#narrow/stream/238009-t-compiler.2Fmeetings
6167

0 commit comments

Comments
 (0)