-
Notifications
You must be signed in to change notification settings - Fork 0
feature: using Coral instead #18
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
base: main
Are you sure you want to change the base?
Changes from all commits
5e0ddb9
87aef72
94e3ba3
4f899f4
3761ecc
54394e8
30acdb5
a5a46d1
edddcef
4fc985a
79388ed
fc714c4
b37f809
091c2d3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,6 @@ | ||
using System; | ||
using System.Collections.Generic; | ||
using System.Linq; | ||
using System.Text; | ||
using System.Threading.Tasks; | ||
using StarEngine; | ||
|
||
using StarEngine; | ||
|
||
namespace Sandbox | ||
namespace StarEngine; | ||
{ | ||
Comment on lines
+1
to
+3
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix circular namespace reference. There's a circular reference issue: the file imports Based on the project structure, this should probably be: using StarEngine;
-namespace StarEngine;
+namespace Sandbox; The Camera class should be in the Sandbox namespace since it's part of the SandboxProject, not the core StarEngine. 🤖 Prompt for AI Agents
|
||
public class Camera : Entity | ||
{ | ||
|
@@ -45,6 +39,5 @@ void OnUpdate(float ts) | |
translation += velocity * ts; | ||
Translation = translation; | ||
} | ||
|
||
} | ||
} |
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Verify Coral include directory variable definition.
The Coral include directory reference looks correct, but ensure that
%{IncludeDir.Coral}
is properly defined in the dependencies configuration.🏁 Script executed:
Length of output: 427
🏁 Script executed:
Length of output: 659
🏁 Script executed:
Length of output: 1027
Define Coral paths in Dependencies.lua
Dependencies.lua currently initializes
IncludeDir = {}
and has no entries forCoral
. You need to add both the include‐directory and library‐path for Coral so that%{IncludeDir.Coral}
and%{Library.Coral}
resolve correctly:• File: Dependencies.lua (around line 5)
Update the tables as follows:
After adding these, the references in Sandbox/premake5.lua (and StarEngine/ and StarEditor/) will resolve without errors.
🤖 Prompt for AI Agents