Skip to content

Fix stray end breaking parse of hybrid_systems.jl#139

Closed
luke-kiernan wants to merge 1 commit into
mainfrom
fix/hybrid-systems-stray-end
Closed

Fix stray end breaking parse of hybrid_systems.jl#139
luke-kiernan wants to merge 1 commit into
mainfrom
fix/hybrid-systems-stray-end

Conversation

@luke-kiernan

Copy link
Copy Markdown
Collaborator

src/hybrid_system_models/hybrid_systems.jl currently fails to parse on main, breaking precompilation of PowerOperationsModels.

The ReserveUp _emit_coverage_constraint! method has an extra end: the if/else already closes (line 1618) and the function closes (line 1621), leaving a stray end at line 1619.

        end      # closes `for`
    end          # closes `if/else`
    end          # ← stray, breaks parse
    return
end              # closes function

Introduced by the hybrid systems port (#104, merged via #105). Removing the stray end makes the file parse and the package precompile again.

🤖 Generated with Claude Code

The ReserveUp `_emit_coverage_constraint!` method had an extra `end`
(the `if/else` already closes, then the function closes), which made
src/hybrid_system_models/hybrid_systems.jl fail to parse and broke
precompilation of PowerOperationsModels on main. Introduced by the
hybrid systems port (#104).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@luke-kiernan

Copy link
Copy Markdown
Collaborator Author

Also addressed in #138 so I guess I'll close this

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.

1 participant