build: one tsconfig, shared oxfmt, fallow, and unmerged generated files - #27
Merged
Merged
Conversation
The shared config sets noEmit, so a separate __tests__/tsconfig.json kept nothing apart. The root now includes __tests__ and vitest.config.ts, without the snapshot folder, and `just typecheck` covers the fixtures, which retires the typecheck-fixtures recipe and its CI step. Co-Authored-By: LLM <noreply@block65.dev>
It duplicated __tests__/fixtures/openai.json and nothing read it. Co-Authored-By: LLM <noreply@block65.dev>
.oxfmtrc.json repeated @block65/shared-config's base options. The repository adds only the snapshot folder to its ignore patterns. Co-Authored-By: LLM <noreply@block65.dev>
Configured from @block65/shared-config. js-yaml is kept, as the Makefile runs its CLI, and the unused @block65/custom-error devDependency goes. Co-Authored-By: LLM <noreply@block65.dev>
A clash in a fixture or snapshot keeps our side without conflict markers, and GitHub collapses them in pull request diffs. Regenerating resolves it. Co-Authored-By: LLM <noreply@block65.dev>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tooling only, with no change to generated output.
noEmit, so__tests__/tsconfig.jsonand thetypecheck-fixturesrecipe and CI step go.@block65/shared-config/oxfmt. The repository adds only the snapshot folder to its ignore patterns.fallow dead-coderuns injust checkand CI. The unused@block65/custom-errordevDependency goes, andjs-yamlstays because the Makefile runs it.-merge linguist-generated. A clash keeps our side without markers and is resolved by regenerating. GitHub collapses these files in diffs.openai.json.First of three stacked PRs. Next: the generated-type fixes, then sequential media.