Zixuan Li

icon indicating a website link https://pig208.itch.io/

icon location New York City

Results 238 comments of Zixuan Li

Systematically survey message content for unimplemented features.

Pushed some typo fixes.

Systematically survey message content for unimplemented features.

Still needs to support: > Run it on all public messages in realms that [publicly list themselves as open communities](https://zulip.com/communities/). For example, have the script sign up a test user...

Systematically survey message content for unimplemented features.

Yeah, it should be straightforward.

Systematically survey message content for unimplemented features.

TODO: ~~Add this to `tools/check` making it easier to run.~~

Systematically survey message content for unimplemented features.

In this revision, I moved to using `zuliprc` to make it easier to run it with `tools/check parser` without specifying additional arguments. The new suite is not included in `tools/check...

Systematically survey message content for unimplemented features.

For reviewing this, it might be helpful to test ~~`tools/check parser`~~ `tools/content/check-features` and see if you can get it working without needing to look at the commit message. Because it...

Systematically survey message content for unimplemented features.

The public message history from CZO can be downloaded from [this protected dropbox link](https://www.dropbox.com/scl/fi/5ugemicgz6lwrsxscqvwk/corpuses.zip?rlkey=xdje3piu5scypt3glc0n4lbrz&st=muzlmp6t&dl=0). We probably will think of a better way of reusing this in the future.

Systematically survey message content for unimplemented features.

Thanks for the review! The PR has been updated.

Systematically survey message content for unimplemented features.

Thanks for the review! I have updated the PR skipping the `_walk` refactor, which conceptually sounds like a good practice to be aware of.

Support saved snippets

Oh right. Thanks! Pushed an update. I realized that I didn't make this clear — let's get #1511 reviewed first, since that's the prerequisite of this PR.