Elixir 1.18 compatibility and quality cleanup#4
Open
nshkrdotcom wants to merge 4 commits intomainfrom
Open
Conversation
- Add @moduledoc false to internal modules to suppress missing doc warnings - Alphabetize alias statements to satisfy Elixir 1.18 compiler checks - Refactor nested conditionals into extracted helper functions for clarity - Replace Enum.map + Enum.join with Enum.map_join for idiomatic Elixir - Use pattern matching on boolean guards instead of if/else branches - Fix TypeScript ash_rpc nullable union type inference with NonNullable wrappers - Export internal TypeScript types for external consumption - Update README with Node.js 18+ requirement and npm install instructions - Bump ash, ash_postgres, ash_sql, ash_typescript, llm_db, and req_llm deps
- Normalize error/runner clauses and config defaults - Align GitHub Issue Bot action modules with linted patterns - Reformat LiveView components and templates
Collaborator
Author
|
It's screwed up for now bc I had path refs. Will clean it up and force push. This is painful bc the deps need to be merged first. They should have PR's |
mikehostetler
added a commit
that referenced
this pull request
Feb 14, 2026
…import Story-File: specs/stories/04_github_integration_and_repo_import.md
pcharbon70
added a commit
that referenced
this pull request
Feb 25, 2026
Phase 1: Add CodeServer facade + project scope adapter
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
1 participant
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.
Summary
Testing