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

updating recent publications #2825

Merged
merged 3 commits into from
Nov 21, 2024
Merged

updating recent publications #2825

merged 3 commits into from
Nov 21, 2024

Conversation

scottcain
Copy link
Collaborator

Note that there was some reformatting introduced by neovim. If that's annoying, I can probably undo it.

@bgruening
Copy link
Member

Can you have a look at the failing tests? We should stay consistent about our YAML styles, which one we take I don't care much.

@scottcain
Copy link
Collaborator Author

@bgruening is that what the problem is (yaml style)? It was unclear from the error messages I was seeing. I'll try repairing it and committing again.

@dannon
Copy link
Member

dannon commented Nov 20, 2024

@scottcain The test failure is unrelated to the yaml formatting, but it's something we can fix. I'm opening a PR that'll just autoformat yaml a single consistent way (with yarn run format) across the codebase and then we don't ahve to worry about any inconsistency of local editors.

@dannon dannon merged commit b141d9e into master Nov 21, 2024
2 checks passed
@bgruening bgruening deleted the scottcain-update-pubs branch November 21, 2024 06:59
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.

3 participants