Mailman 3 December 2020 - yt-dev

Change default branch name
by Matthew Turk Jan. 20, 2021

Jan. 20, 2021

Hi folks, In January, Github is making it easier to change the default branch name with a toggle on the website: https://github.com/github/renaming I know lots of other projects have done this (including maestro) and we could do it ourselves, but I'd like to propose that we do this as soon as GH has the tooling in place to handle it for us. If anyone objects we can dig into this during a team meeting or something, but personally this kind of seems uncontroversial and should be easy to implement. -Matt

6 14

Auto-fixing linting errors
by Clément Robert Jan. 11, 2021

Jan. 11, 2021

Dear yt-dev I recently found out about https://pre-commit.ci, and I'd like to propose we adopt this tool as part of our CI for linting/auto-fixing. The idea is that this automatically runs $ pre-commit run --all-files on each pull request and commits to the corresponding branch to auto-fix errors with pre-commit hooks (black, isort and flynt in our case). I'm reaching out to you because this would have small repercusions on everyone's workflow. Let me try to give a comprehensive description of gains and costs in the following. what it gets us a simplification of our CI tooling, namely: - `tests/lint_requirements.txt` - `.github/workflows/style-checks.yaml` - https://github.com/yt-project/slash-command-processor all become useless `.github/PULL_REQUEST_TEMPLATE.yaml` will also be simplified as all linting items in `PR Checklist` become redundant. On top of this, new hooks could be seamlessly added in the future without increasing the entry barrier for occasional contributors (as installing pre-commit hooks locally is and remains an opt-in). what it costs us - "zero" configuration: the only requirement is `.pre-commit.yaml`, which is already part of the repo. - minor updates to .pre-commit-config.yaml, namely : - bumping black's version (reason: black's `--exclude` flag is overidden by pre-commit's `--all-files`, but more recent versions of black have a `--force-exclude` flag that overcomes this) this will require a new PR to adopt black's latest (minor) updates in code styling - signing up tohttps://pre-commit.ci (free) what this changes to your workflow PRs will be automatically fixed by the bot, so you'll either need to occasionally run `git pull` or `git push --force` to account for it. Note that this can be avoided completely by installing pre-commit hooks locally which is and will remain an opt-in, see https://yt-project.org/docs/dev/developing/developing.html#pre-commit-hooks. Let me know what you think. Take care, and happy new year ! Cheers, Clément

3 3

proposal: move new blog format into yt-project org
by Madicken Munk Dec. 24, 2020

Dec. 24, 2020

Hi yt-dev, I have been working on an update to our blog, which is currently on the yt-project.org website here: https://blog.yt-project.org/ as you can see, there haven't been updates to it in quite some time. Matt and I have ported all of the old posts from this blog to a hugo page that has tagging and categorization, so one can filter blog posts based on search interest. Here's what it looks like right now: munkm.github.io/yt-blog/ and here is the repo currently: https://github.com/munkm/yt-blog What do you all think? I think we could get some very cool contributions to the blog this way. -Madicken PS: I presented this blog at the yt user's workshop a few weeks ago! If you want to see it in action you can watch it here: https://www.youtube.com/watch?v=wKh_8AAlFhY

3 2

Proposal: move two repositories to yt-project
by Matthew Turk Dec. 22, 2020

Dec. 22, 2020

Hi folks, For a while, we've had two projects "incubating" in the org data-exp-lab, which is tied to the research group I'm in here at UIUC. These are the "widgyts" repo, which has some jupyter widgets in it, and "yt_idv" which is where the IDV now lives. I'd like to propose moving these to yt-project. I don't see this as being very controversial, but I thought I'd give the opportunity for folks to chime in yay/nay. Thanks, Matt

4 4