Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More work on the monthly update post #300

Merged
merged 12 commits into from
Feb 28, 2025
Merged

Conversation

lqd
Copy link
Member

@lqd lqd commented Feb 28, 2025

Here's the bunch of fixes I mentioned when reviewing the other PRs for #297:

  • pluralization of the update counts
  • compute the goal counts
  • use the date for the updates instead of the full timestamp
  • sort the goal updates
  • some visual separation between flagship goals (and goals that need help): whenever there's text, markdown visually lumps together the updates <details> with the previous goal -- mostly a matter of taste, we can tweak/remove it if you don't like it. This weirdness may also appear when goal updates have tl;drs as well...
  • cleanups to ignore: warnings, unused files (for the llm, we can probably merge/remove a bunch of crates soon), rustfmt everything

You can check the outcome of these changes here.

Rendered

@lqd lqd requested a review from nikomatsakis February 28, 2025 10:56
@lqd lqd mentioned this pull request Feb 28, 2025
@spastorino spastorino merged commit a509563 into rust-lang:main Feb 28, 2025
2 checks passed
@lqd lqd deleted the monthly-post branch February 28, 2025 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants